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 #9500 (closed)

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

Bug: Inject host/port into PublicRepositoryI

Reported by: cblackburn Owned by: jamoore
Priority: critical Milestone: OMERO-4.4.7
Component: OmeroFs Version: n.a.
Keywords: n.a. Cc: cxallan, jamoore
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2012-10-23 (1)

Description

The host and port are hard-coded in the importMetadata method. There's even a nice TODO for it to be fixed but not ticket was raised at the time. Now it needs doing!

Change History (6)

comment:1 Changed 12 years ago by cblackburn

  • Milestone changed from OMERO-4.4.2 to Future
  • Owner set to cblackburn
  • Sprint 2012-08-28 (3) deleted

This bug is only in the fs-lite import code and so not relevant to 4.4.2. I've moved the ticket to future for now.

comment:2 Changed 12 years ago by jmoore

  • Milestone changed from Future to OMERO-4.4.x

Moving to 4.4.x (soon to be 4.5.0) with the rest of the fs-lite stuff.

comment:3 Changed 12 years ago by jmoore

Referencing ticket #909 has changed sprint.

comment:4 Changed 12 years ago by jmoore

  • Owner changed from cblackburn to jmoore
  • Remaining Time set to 0.1
  • Sprint set to 2012-10-23 (1)
  • Status changed from new to accepted

Fixed via Blitz Registry on my fslite branch.

comment:5 Changed 12 years ago by jmoore

  • Remaining Time changed from 0.1 to 0
  • Resolution set to fixed
  • Status changed from accepted to closed
commit 19b4bbf3e25622301676f4a258d860e432972c3a
Author: jmoore <josh@glencoesoftware.com>
Date:   Sun Sep 30 15:29:19 2012

    Refactor hard-coded import connection (Fix #9500)
    
    Rather than passing a server and port we connect now
    to the server manager internally and retrieve the
    existing ServiceFactoryPrx. A future version may want
    to create a separate session for the import process so
    that if the client exists the process can still complete.

Haven't opened a PR yet since 383 hasn't been merged. Will do so asap.

comment:6 Changed 12 years ago by jmoore <josh@…>

(In [7ae0ea675ad8cc4a6489f7c92587d1dc987e7044/ome.git] on branch develop) Refactor hard-coded import connection (Fix #9500)

Rather than passing a server and port we connect now
to the server manager internally and retrieve the
existing ServiceFactoryPrx?. A future version may want
to create a separate session for the import process so
that if the client exists the process can still complete.

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.68957 sec.)

We're Hiring!