Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 414907 - support storing multiple sets of credentials for the same repository
Summary: support storing multiple sets of credentials for the same repository
Status: RESOLVED WONTFIX
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: PC All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Sam Davis CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 278474
  Show dependency tree
 
Reported: 2013-08-12 14:57 EDT by Sam Davis CLA
Modified: 2014-02-17 19:59 EST (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sam Davis CLA 2013-08-12 14:57:53 EDT
As part of the move to the secure store, we should support storing multiple sets of credentials for the same repository, e.g. by indexing them using the username.
Comment 1 Sam Davis CLA 2014-01-03 18:51:56 EST
This is needed to support users with multiple workspaces using different credentials in each workspace.
Comment 2 Sam Davis CLA 2014-02-04 17:43:11 EST
I've pushed a WIP review but it doesn't quite work. It uses the username as part of the key to look up passwords, which is fine. The problem is that we can't support storing multiple different usernames in the secure store unless each TaskRepository uses a random value as the key and stores that key in it's properties. The other option would be to store all usernames as properties on TaskRepository and not in the secure store.

The problem with both approaches is that the URL is no longer enough to look up the credentials. Given that storing multiple sets of credentials is only needed when using multiple workspaces and it's easy to use a different secure storage for each one, I'm not sure it's worth doing this.

21539: [WIP] 414907: support storing multiple sets of credentials for the same repository [I2c78b225]
https://git.eclipse.org/r/#/c/21539/
Comment 3 Sam Davis CLA 2014-02-17 19:59:30 EST
Marking WONTFIX as per reasoning above.