Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 387094

Summary: [client] Save Git credentials: I - Uncrypted local storage
Product: [ECD] Orion Reporter: Maciej Bendkowski <maciej.bendkowski>
Component: GitAssignee: Maciej Bendkowski <maciej.bendkowski>
Status: CLOSED DUPLICATE QA Contact:
Severity: normal    
Priority: P3 CC: malgorzata.tomczyk, Szymon.Brandys
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows 7   
Whiteboard:
Bug Depends on:    
Bug Blocks: 359952    
Attachments:
Description Flags
Crash look after firebug init none

Description Maciej Bendkowski CLA 2012-08-13 04:43:24 EDT 
First part of the bug 359952 solution.

We would like to introduce a credentials storage mechanism, which may facilitate working with git repositories. The private key would be stored in the local storage and used automatically to authenticate the git user. If the private key would not be present or invalid, the user would be asked to upload the key from a local file. At this point, we will not introduce any encryption.
Comment 1 Maciej Bendkowski CLA 2012-08-16 06:44:55 EDT 
Created attachment 219951 [details]
Crash look after firebug init

After initiating Firebug the header renders even stranger (see attachment).
Comment 2 Maciej Bendkowski CLA 2012-08-16 11:27:21 EDT 
Fix: https://github.com/maciej-bendkowski/orion.client/commit/c63df388e51b8e3b3be420a890259df20fce4be0

This fix provides some basic functionality. Git private key may be uploaded from a local file and if given, it's stored in the local storage/cookies. After this process the user is automatically authenticated with the stored private key. If any authentication error occurs the key is erased. At the moment it's not possible to switch back to manual ssh credentials gathering if the private key was already provided. We plan to allow switching from auto to manual mode (and back) by adding a custom user preference in settings like "Clear credentials". For development/testing purposes I recommend clearing the local storage (or cookies).
Comment 3 Maciej Bendkowski CLA 2012-08-28 11:13:35 EDT 

*** This bug has been marked as a duplicate of bug 387878 ***