Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remember authentication details with endpoint configuration #1067

Closed
1 task
nikku opened this issue Dec 11, 2018 · 4 comments · Fixed by #1526
Closed
1 task

Remember authentication details with endpoint configuration #1067

nikku opened this issue Dec 11, 2018 · 4 comments · Fixed by #1526
Assignees
Labels
Milestone

Comments

@nikku
Copy link
Member

nikku commented Dec 11, 2018

Is your feature request related to a problem? Please describe.

As a user I am frustrated if I need to re-configure the authentication details every time I re-deploy a diagram.

Describe the solution you'd like

  • Deployment details, including authentication type (+ user name) (+ credentials ?) are stored within the user data config.json upon successful deployment and restored on deploy dialog re-open.

Describe alternatives you've considered

Don't store the details (password / token) if we don't find a way to secure it.

@barmac
Copy link
Collaborator

barmac commented Sep 23, 2019

I had a look at how VS Code Docker Extension handles the passwords and noticed that:

  • the passwords are stored in the system's keychain using https://github.com/atom/node-keytar
  • once the credentials are used they are kept in a session cache to not overuse the keychain

@barmac
Copy link
Collaborator

barmac commented Oct 1, 2019

Idea: Checkbox for remembering details

@barmac
Copy link
Collaborator

barmac commented Oct 1, 2019

Idea: Remember auth per endpoint url and the auth type.

@bpmn-io-tasks bpmn-io-tasks bot added in progress Currently worked on and removed ready Ready to be worked on labels Oct 9, 2019
barmac added a commit that referenced this issue Oct 10, 2019
@bpmn-io-tasks bpmn-io-tasks bot added needs review Review pending and removed in progress Currently worked on labels Oct 10, 2019
@nikku nikku modified the milestones: M31, M32 Oct 15, 2019
@nikku nikku added ready Ready to be worked on and removed needs review Review pending labels Oct 15, 2019
@nikku
Copy link
Member Author

nikku commented Oct 15, 2019

Removed from #1439. We'll ship this with a future version of the tool.

barmac added a commit that referenced this issue Oct 21, 2019
barmac added a commit that referenced this issue Oct 21, 2019
@barmac barmac added needs review Review pending in progress Currently worked on and removed ready Ready to be worked on needs review Review pending labels Oct 21, 2019 — with bpmn-io-tasks
@nikku nikku added ready Ready to be worked on and removed in progress Currently worked on labels Oct 22, 2019 — with bpmn-io-tasks
nikku pushed a commit that referenced this issue Oct 31, 2019
nikku pushed a commit that referenced this issue Nov 5, 2019
nikku pushed a commit that referenced this issue Nov 15, 2019
nikku pushed a commit that referenced this issue Nov 18, 2019
nikku pushed a commit that referenced this issue Nov 19, 2019
nikku added a commit that referenced this issue Nov 22, 2019
* remember authentication details, if the user actively
  decides this
* restore global endpoint configuration, as intended until
  it is properly addressed via #804
* refactor deployment tool, pulling out validation into separate
  component
* pull out menu update functionality to be re-added at a later
  point in time, in a different place
* deployment tool UI remains untouched

Closes #1067
nikku added a commit that referenced this issue Nov 22, 2019
* remember authentication details, if the user actively
  decides this
* restore global endpoint configuration, as intended until
  it is properly addressed via #804
* refactor deployment tool, pulling out validation into separate
  component
* pull out menu update functionality to be re-added at a later
  point in time, in a different place
* deployment tool UI remains untouched

Closes #1067
nikku added a commit that referenced this issue Nov 22, 2019
* remember authentication details, if the user actively
  decides this
* restore global endpoint configuration, as intended until
  it is properly addressed via #804
* refactor deployment tool, pulling out validation into separate
  component
* pull out menu update functionality to be re-added at a later
  point in time, in a different place
* deployment tool UI remains untouched

Closes #1067
nikku added a commit that referenced this issue Nov 22, 2019
* remember authentication details, if the user actively
  decides this
* restore global endpoint configuration, as intended until
  it is properly addressed via #804
* refactor deployment tool, pulling out validation into separate
  component
* pull out menu update functionality to be re-added at a later
  point in time, in a different place
* deployment tool UI remains untouched

Closes #1067
@nikku nikku added needs review Review pending and removed ready Ready to be worked on labels Nov 22, 2019 — with bpmn-io-tasks
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Nov 22, 2019
pinussilvestrus pushed a commit that referenced this issue Nov 22, 2019
* remember authentication details, if the user actively
  decides this
* restore global endpoint configuration, as intended until
  it is properly addressed via #804
* refactor deployment tool, pulling out validation into separate
  component
* pull out menu update functionality to be re-added at a later
  point in time, in a different place
* deployment tool UI remains untouched

Closes #1067
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants