Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename ?server query parameter to ?host #414

Merged
merged 1 commit into from
Aug 15, 2022

Conversation

will-moore
Copy link
Member

@will-moore will-moore commented Jul 5, 2022

This prevents collision with ?server=2 used by webclient @login_required decorator

This testing is the same omero-web functionality as recently tested in ome/omero-web#386, that was previously broken because iviewer uses the ?server=2 parameter from the URL which causes errors (this parameter was originally added for dev functionality but not used now). This has been re-named to host to prevent the collision, allowing the omero-web workflow to function as normal...

See ome/omero-web#379

To test, get a session ID, e.g via cli to e.g. nightshade

$ omero login
$ omero sessions list

Try to login to a server that is not the first on the list. E.g. for merge-ci, try to login to nightshade, listed 3rd (server=3)
First you need to look up an Image ID on target server (nightshade) that you can access with the session created above.

Then go to:

https://merge-ci.openmicroscopy.org/web/webclient/img_detail/IMAGE_ID/?bsession=SESSION_ID&server=3

You should be able to see the image.

This prevents collision with ?server=2 used by webclient @login_required decorator
@will-moore
Copy link
Member Author

will-moore commented Jul 6, 2022

Today on merge-ci I'm seeing Unhandled rejection Error: BindingLanguage must implement inspectTextContent().
errors in inviewer - Googling found https://discourse.aurelia.io/t/bindinglanguage-must-implement-inspecttextcontent/1967 and aurelia/skeleton-navigation#759 suggesting a clean node_modules should fix...

Hard to see how this error is related to this PR...

@joshmoore
Copy link
Member

Is there any chance someone could have been relying on the previous naming? (Despite or because of (??) the collision)

@will-moore
Copy link
Member Author

Ah, I see #413 dependabot[bot] 'Bump aurelia-framework from 1.1.4 to 1.4.1' got created yesterday, which is a more likely candidate for these issues!

@will-moore
Copy link
Member Author

@joshmoore This is not documented anywhere that I can see, and it's not a feature that I've ever used (or even been aware of till now), and I still haven't understood the need for it.
This code e.g. 6dfa178 was added before the dev config at 7348ec4#diff-03651daa986b59c8421cba91a0ff6e7ffdfc40623e571f6525f648c2a6e19308R59 which is what I've been using to achieve the same thing (point iviewer dev server at your localhost).
So I think it's unlikely that someone is using it.

@pwalczysko
Copy link
Member

Tested on workshop (server=4). All worked fine.

@jburel jburel merged commit 4ab93b6 into ome:master Aug 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants