Modify

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)

npm_patch (16.5 KB ) - added by bastiK 16 years ago.
npm_patch_r1953 (17.4 KB ) - added by bastiK 16 years ago.
npm_add_osm_server (11.5 KB ) - added by bastiK 16 years ago.
make not only password and username part of credentials management, but also the osm server url

Download all attachments as: .zip

Change History (12)

by bastiK, 16 years ago

Attachment: npm_patch added

comment:1 by stoecker, 16 years ago

Owner: changed from team to bastiK
Status: newneedinfo

The code got a bit out of sync. Could you please update it to current SVN.

by bastiK, 16 years ago

Attachment: npm_patch_r1953 added

comment:2 by bastiK, 16 years ago

sure

comment:3 by stoecker, 16 years ago

Resolution: fixed
Status: needinfoclosed

(In [1955]) close #3112 - patch by bastiK - rework username/password handling to allow password hooks

comment:4 by stoecker, 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 bastiK, 16 years ago

Resolution: fixed
Status: closedreopened

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 stoecker, 16 years ago

Owner: changed from bastiK to team
Status: reopenednew
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 bastiK, 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 bastiK, 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 stoecker, 16 years ago

Summary: hook for new plugin (native password manager)[PATCH] hook for new plugin (native password manager)

comment:9 by stoecker, 16 years ago

Resolution: fixed
Status: newclosed

(In [2015]) fixed #3112 - patch by bastiK - cleanup password hook

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.