Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #12228 (closed)

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

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)

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.64317 sec.)

We're Hiring!