Task #12228 (closed)
Bug: CLI interactive login ignores port in server specification
Reported by: | khgillen | Owned by: | jamoore |
---|---|---|---|
Priority: | major | Milestone: | 5.0.2 |
Component: | General | Version: | 5.0.1 |
Keywords: | n.a. | Cc: | ux@… |
Resources: | n.a. | Referenced By: | n.a. |
References: | n.a. | Remaining Time: | 0.0d |
Sprint: | n.a. |
Description
Hi,
Logging into an OMERO 5.0.1 server with
bin/omero login
What I would call an Interactive login, where OMERO will prompt for the connection details, as outlined in the docs "If no argument can be specified, the interface will ask for the connection credentials" at https://www.openmicroscopy.org/site/support/omero5/users/command-line-interface.html
Server: [localhost]localhost:34064 Username: [khgillen] Password: Created session guid (khgillen@localhost:4064). Idle timeout: 10.0 min. Current group: default
As you can see, I've been logged into the default port, despite my port appended to the hostname. I feel that although we could update the docs to represent what actually happens, if the port is parsed by the following:
$ bin/omero login username@servername:4064
then it should also be parsed at an 'interactive' login.
Change History (3)
comment:1 Changed 10 years ago by jamoore
- Resolution set to fixed
- Status changed from new to closed
comment:2 Changed 10 years ago by jmoore <josh@…>
- Remaining Time set to 0
(In [b41d59a606290ea90119a2fe012b19b86a164c23/ome.git] on branch develop) Fix port on interactive login (Fix #12228)
Primary cause was a typo "prt" (which would have been
detected by flake8!) While trying to track this down,
I went ahead and added a more proper regex for the
server string parsing.
comment:3 Changed 10 years ago by Sébastien Besson <seb.besson@…>
(In [87f0a45cb0d3b0c949d7f82f26877056e95aff75/ome.git] on branch develop) Merge pull request #2360 from joshmoore/rebased/develop/12228-cli-login
Fix port on interactive login (Fix #12228) (rebased onto develop)
PR opened https://github.com/openmicroscopy/openmicroscopy/pull/2358