Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 186379 - Determine repository URL on validation
Summary: Determine repository URL on validation
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P2 enhancement (vote)
Target Milestone: 2.1   Edit
Assignee: Steffen Pingel CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 202834
  Show dependency tree
 
Reported: 2007-05-10 10:56 EDT by Steffen Pingel CLA
Modified: 2007-09-18 00:23 EDT (History)
1 user (show)

See Also:


Attachments
mylyn/context/zip (5.93 KB, application/octet-stream)
2007-09-16 02:38 EDT, Steffen Pingel CLA
no flags Details
location dialog (12.69 KB, image/png)
2007-09-16 15:53 EDT, Steffen Pingel CLA
no flags Details
mylyn/context/zip (12.56 KB, application/octet-stream)
2007-09-16 15:54 EDT, Steffen Pingel CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Steffen Pingel CLA 2007-05-10 10:56:01 EDT
A JIRA server sends the repository location as a redirect on login. This location should be used as the repository URL in the properties dialog when the connection is validated.
Comment 1 Steffen Pingel CLA 2007-09-14 21:52:02 EDT
- do not follow redirect to http from https without warning
Comment 2 Steffen Pingel CLA 2007-09-16 02:38:40 EDT
While implementing this I could not quite figure out what the best user interaction when validating a repository that redirects to another location (e.g. https:// instead of http://):

 1) show a selection dialog displaying alternative locations ("Keep my location", "Use redirect location")
 2) replace url in dialog and display a warning [current implementation]
Comment 3 Steffen Pingel CLA 2007-09-16 02:38:45 EDT
Created attachment 78513 [details]
mylyn/context/zip
Comment 4 Steffen Pingel CLA 2007-09-16 15:53:58 EDT
Created attachment 78518 [details]
location dialog

I have implemented a dialog that prompts the user if the repository location does not match (see attached snap shot). We can iterate for the next release how to improve the interaction.

I have also changed the logging of login errors to only be active when the repository is being validated to avoid polluting the Eclipse error log.
Comment 5 Steffen Pingel CLA 2007-09-16 15:54:01 EDT
Created attachment 78519 [details]
mylyn/context/zip
Comment 6 Steffen Pingel CLA 2007-09-16 15:57:14 EDT
I have changed the wording to "Keep current location:".

We should consider making this part of the repository settings API. Bugzilla displays a modal error dialog when attempting to validate http://bugs.eclipse.org/bugs instead of following the redirect to https://...

Marking as resolved.

Comment 7 Eugene Kuleshov CLA 2007-09-16 20:10:02 EDT
It is somehow weird to have those radio buttons. Maybe we could rework dialog to show urls in the message, but use buttons for taking particular actions, like update, ignore and cancel.
Comment 8 Steffen Pingel CLA 2007-09-16 20:20:45 EDT
There could be potentially three urls:

- the user provided one
- the one returned by getServerInfo().getBaseURL()
- the one the redirects during web login pointed to

We could replace the radio buttons by a combo box. Another idea I had was to add them to the combo box in the repository settings dialog but I'd rather prioritize other bugs and push this to the next release.
Comment 9 Mik Kersten CLA 2007-09-17 21:54:11 EDT
+1 for the prioritization suggested in comment#8.

I actually prefer the radio buttons to a combo box because they show the entire text for both options.  Fyi in Vista this kind of UI is standard and I find it works very well.  But instead of Radios they make large hyperlink-style buttons with text that you can click, reducing the click count from 2 to 1.
Comment 10 Eugene Kuleshov CLA 2007-09-17 23:33:03 EDT
Mik, for the sake of God, please don't add hyperlinks for stuff like that. Double-clickable radio buttons are old ui trick that been probably used since NT times...
Comment 11 Mik Kersten CLA 2007-09-18 00:23:19 EDT
No plans to do so,  iwas just pointing out some similarity with something that is working well in the Vista UI.  Btw, visually they really look like hyperlinks, just act a bit like them (e.g. parts of text underline on hover).