Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 387878 - [client] Save Git credentials: II - Enhance usability
Summary: [client] Save Git credentials: II - Enhance usability
Status: RESOLVED FIXED
Alias: None
Product: Orion
Classification: ECD
Component: Git (show other bugs)
Version: 0.5   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 1.0 M2   Edit
Assignee: Maciej Bendkowski CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 387094 (view as bug list)
Depends on:
Blocks: 359952
  Show dependency tree
 
Reported: 2012-08-23 04:56 EDT by Maciej Bendkowski CLA
Modified: 2012-09-06 06:42 EDT (History)
2 users (show)

See Also:
maciej.bendkowski: review? (Szymon.Brandys)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Maciej Bendkowski CLA 2012-08-23 04:56:17 EDT
Second part of bug 359952 solution.

Based on bug 387094 fix, we would like to enhance the usability of this feature. New components which will be added:

1. Store one private key per repository.
2. Add a checkbox in the git credentials dialog which would enable the storage mechanism.
3. Allow enabling/disabling storage mechanism via user preferences in the settings section.
4. Add a 'Clear credentials' option in the settings section.
5. Add private key import/export options in the user settings section.
Comment 1 Maciej Bendkowski CLA 2012-08-28 03:40:44 EDT
(In reply to comment #0)
> 5. Add private key import/export options in the user settings section.

No import/export is really necessary. Instead, we allow enabling/disabling prompting providing a checkbox for each repository in the user profile.
Comment 2 Maciej Bendkowski CLA 2012-08-28 11:11:51 EDT
Experimental fix: https://github.com/maciej-bendkowski/orion.client/commit/2ea70c19096c053bf156059a60447fbc12b6cb8d

Private key and the passphrase are stored in the local storage. For now, in a persistent manner. Each repository may hold an information whether to prompt for credentials or not (besides a global switch). This can be controlled in the git credentials dialog and in the user settings in the user profile category. At the moment this version in quite experimental, so any suggestions on workflow, view, etc. are most welcome.
Comment 3 Maciej Bendkowski CLA 2012-08-28 11:13:35 EDT
*** Bug 387094 has been marked as a duplicate of this bug. ***
Comment 4 Szymon Brandys CLA 2012-09-03 09:31:48 EDT
Comments for get credentials dialog
1) "Don't prompt me again" should be visible only if 'Git Credentials" are enabled
2) I would put "Don't prompt me again" at the bottom of the dialog (between the content and OK button)
3) Credentials are saved only if I use "Browse...", if Put them in the text area they are not saved
4) Since "Don't prompt me again" applies only to private key, maybe it should be disabled if "Ssh password" is selected

Comments for changes in "User Profile"
1) I would move "Git Credentials" to its own section
2) "Clear Git Credentials" would be a section action then
Comment 8 Szymon Brandys CLA 2012-09-06 06:42:46 EDT
The fix merged to master.