Opened 16 years ago
Closed 16 years ago
#3112 closed enhancement (fixed)
[PATCH] hook for new plugin (native password manager)
Reported by: | bastiK | Owned by: | team |
---|---|---|---|
Priority: | major | Milestone: | |
Component: | Core | Version: | latest |
Keywords: | Cc: |
Description
The patch moves the osm-password related code to a central static object. This allowes the upcoming plugin for gnomekeyring and KWallet support to provide its own password handling backend.
So far this shouldn't change any visible behaviour.
Attachments (3)
Change History (12)
by , 16 years ago
comment:1 by , 16 years ago
Owner: | changed from | to
---|---|
Status: | new → needinfo |
by , 16 years ago
Attachment: | npm_patch_r1953 added |
---|
comment:3 by , 16 years ago
Resolution: | → fixed |
---|---|
Status: | needinfo → closed |
comment:4 by , 16 years ago
One note regarding future extensions - Handling the server URL in the password mechanism may be a good idea as well, so different passwords could be managed for test and development servers.
Also the openstreetbugs plugin needs password management :-)
comment:5 by , 16 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
You can have two .josm folders and run
$ java -Djosm.home=~/.josm_dev -jar josm-custom.jar
Isn't this more straightforward than complicated GUI?
And for openstreetbugs: I couldn't find anything like registration or so. Maybe this is going to change?
comment:6 by , 16 years ago
Owner: | changed from | to
---|---|
Status: | reopened → new |
Summary: | [PATCH] hook for new plugin (native password manager) → hook for new plugin (native password manager) |
I don't know what is best. But password managers can manage different password sets, so it seems a sensible idea to me :-)
by , 16 years ago
Attachment: | npm_add_osm_server added |
---|
make not only password and username part of credentials management, but also the osm server url
comment:7 by , 16 years ago
Ok, in one way or another it might be a good idea to include the server url.
Here we go.
comment:8 by , 16 years ago
Summary: | hook for new plugin (native password manager) → [PATCH] hook for new plugin (native password manager) |
---|
comment:9 by , 16 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
The code got a bit out of sync. Could you please update it to current SVN.