Community
Participate
Working Groups
I've had my plugin go missing from the installed list many times now and I think I've found the scenario. I'm using https://fbug.googlecode.com/svn/extensions/dyne/branches/dyne0.1 linked to |dyne| in orion. I don't think this matters, but... Start with a file to edit http://localhost:8080/edit/edit.html#/file/c/doc/dynedoc.css and a user-defined plugin installed. I don't know if it matters but my plugin is self hosted: http://localhost:8080/file/c/content/orionPlugin/firebugConnection.html Save the URL for the file to edit. Logout of Orion. Open the edit URL. The page loads but the plugins do not. Edit the file and Save. You get a login dialog: log in. Your edits are saved, but the plugin has vanished.
Well it seems like the logon has to expire. I can't reproduce it by logging out.
It sounds to me like your plugin is being protected by the workspace authentication filter. If that's true then I bet you're getting a 401 in your plugin and we're not being good about handling it. The authentication story for plugins is not at all where we want it and at the moment either piggybacking on existing authentication tokens or else only works with anonymous. If your session timed out then that indeed would hose you.
I going to use this bug to track ensuring I add the login bits to our soon to be done notification area.
(In reply to comment #3) > I going to use this bug to track ensuring I add the login bits to our soon > to be done notification area. The recent changes removed handling 401 when xhr calls are made from the page and the session expired. I don't understand why you are removing this code?
Hi Szymon We're mid-stream in the change -- I'm adding it back now but differently and at a lower layer. The UI is different and rather than using the login dialog we're using the status service notification and a "login" link that the plugin provides in its manifest. auth.js has been broken for essentially the entire release as it was involved in a dependency cycle so we were hitting a TypeError rather than letting the error bubble up. No-one really noticed because the bootstrap always redirects to the login page if not logged in. auth.js and logindialog were referenced in roughly 10 files so we decided to remove the references as a first step to help ensure our merge on commit would be simple for a reviewer to look at. More coming shortly...
John reviewed. Its not a beautiful UX which we'll work on in rc3 but its checked in -- also includes cross orion filesystem support so you can mount your local orions filesystem in orion.eclipse.org.
Ken reviewed. I did one final fix to ensure the login window opens in a _blank target and handled an error check better that masking some problem reporting.
