Bug 241716 - Handle change expired password
Summary: Handle change expired password
Status: RESOLVED FIXED
Alias: None
Product: Target Management
Classification: Tools
Component: RSE (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P2 normal (vote)
Target Milestone: 3.0.1   Edit
Assignee: David Dykstal CLA
QA Contact: Martin Oberhuber CLA
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2008-07-22 13:43 EDT by Richie Yu CLA
Modified: 2008-08-25 12:10 EDT (History)
3 users (show)

See Also:


Attachments
updates local variable to handle potential change in .validate () (15.49 KB, patch)
2008-07-23 14:29 EDT, Richie Yu CLA
no flags Details | Diff
updates local variable to handle potential change in .validate () (17.53 KB, patch)
2008-07-23 14:31 EDT, Richie Yu CLA
no flags Details | Diff
updates local variable to handle potential change in .validate () (2.22 KB, patch)
2008-07-23 14:46 EDT, Richie Yu CLA
ddykstal.eclipse: iplog+
Details | Diff
updates local variable to handle potential change in .validate () (15.49 KB, application/octet-stream)
2008-07-23 14:48 EDT, Richie Yu CLA
no flags Details
updates local variable to handle potential change in .validate () (17.53 KB, application/octet-stream)
2008-07-23 14:50 EDT, Richie Yu CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Richie Yu CLA 2008-07-22 13:43:29 EDT
Hi,

We're extending the validator.validate () method to handle changing expired password.  We need to be able to update the local variables in the dialogs that invoke this method.  Could we query the credentials variable after the validate?

Here's the snippet of requested changes:


In StandardCredentialsProvider class, acquireCredentials method

ICredentials credentials = getCredentials(); // new
     if (validator != null) {
	SystemMessage m = validator.validate(credentials); // new
	password = credentials.getPassword();  // new

In SystemPasswordPromptDialog class, processOK method
SystemSignonInformation(hostName, userId, password, systemType);
SystemMessage m = signonValidator.validate(credentials);
password = credentials.getPassword();  //new

Thanks,
Richie
Comment 1 Martin Oberhuber CLA 2008-07-23 10:23:54 EDT
Richie, it looks like you have a pretty good idea of what you want, could you attach a patch here that does what you need? - Please also add a comment indicating that you contribute the patch under EPL, see http://www.eclipse.org/dsdp/tm/development/committer_howto.php#external_contrib
Comment 2 Richie Yu CLA 2008-07-23 14:29:27 EDT
Created attachment 108245 [details]
updates local variable to handle potential change in .validate ()

I, Richie Yu, declare that I developed attached code from scratch, without referencing any 3rd party materials except material licensed under the EPL. {I am authorized by my employer to make this contribution under the EPL.}
Comment 3 Richie Yu CLA 2008-07-23 14:31:00 EDT
Created attachment 108247 [details]
updates local variable to handle potential change in .validate ()

I, Richie Yu, declare that I developed attached code from scratch, without referencing any 3rd party materials except material licensed under the EPL. {I am authorized by my employer to make this contribution under the EPL.}
Comment 4 Richie Yu CLA 2008-07-23 14:46:51 EDT
Created attachment 108250 [details]
updates local variable to handle potential change in .validate ()

I, Richie Yu, declare that I developed attached code from scratch, without referencing any 3rd party materials except material licensed under the EPL. {I am authorized by my employer to make this contribution under the EPL.
Comment 5 Richie Yu CLA 2008-07-23 14:48:21 EDT
Created attachment 108251 [details]
updates local variable to handle potential change in .validate ()
Comment 6 Richie Yu CLA 2008-07-23 14:50:08 EDT
Created attachment 108252 [details]
updates local variable to handle potential change in .validate ()
Comment 7 David McKnight CLA 2008-07-31 11:32:20 EDT
Dave D, this is your area.  Can you deal with this patch?
Comment 8 David Dykstal CLA 2008-08-24 22:36:42 EDT
Applied the patch with minor modifications to comments.
Updated IP log file.
Comment 9 Martin Oberhuber CLA 2008-08-25 11:24:33 EDT
Dave can you please set a bugzilla "iplog+" flag on the attachment which you have actually used for the patch? For me, it looks like this would be the 3rd one and numbers 4 and 5 can be marked obsolete?