Opened 3 years ago

Closed 3 years ago

#2477 closed enhancement (fixed)

wcst_import - check SECORE URL from embedded petascope's port

Reported by: Bang Pham Huu Owned by: Bang Pham Huu
Priority: major Milestone: 10.0
Component: wcst_import Version: 9.8
Keywords: Cc: Dimitar Misev, Vlad Merticariu
Complexity: Medium

Description (last modified by Bang Pham Huu)

In case, petascope runs in embedded port 9009 and SECORE runs in embedded port 9010 (in a system configured before)
with secore_urls=http://localhost:9010/def in petascope.properties.

As def.war is now not deployed anymore as SECORE is a part of petascope as def.jar, wcst_import should check SECORE from embedded petascope as well by getting the port 9009 from service_url setting of ingredients files.

NOTE: if secore url is localhost (regardless of what port) and it doesn't exist on this port (e.g. 9010), then assume it's internal on service url (e.g: http://localhost:9009/rasdaman/def) and print a warning to fix it in petascope.properties.

Change History (5)

comment:1 by Dimitar Misev, 3 years ago

Why does it need to get port 9009 from petascope.properties, isn't the endpoint in the ingredients file?

Either way, it must print a warning, it shouldn't silently change the URL.

comment:2 by Bang Pham Huu, 3 years ago

Why does it need to get port 9009 from petascope.properties, isn't the endpoint in the ingredients file?

No, SECORE endpoint doesn't need to exist in the ingredients files. In this case, wcst_import checks SECORE URLs from petascope.properties (that was your idea long time ago) and uses it.

it shouldn't silently change the URL

But, you said "assume it's "internal" " then it is still like what I wrote in the ticket description.

if secore url is localhost (regardless of what port) then assume it's "internal" + print a warning to fix it in petascope.properties?

in reply to:  2 comment:3 by Dimitar Misev, 3 years ago

Replying to Bang Pham Huu:

No, SECORE endpoint doesn't need to exist in the ingredients files. In this case, wcst_import checks SECORE URLs from petascope.properties (that was your idea long time ago) and uses it.

I meant the petascope endpoint.. as you said "wcst_import should check SECORE from embedded petascope as well by getting the port 9009 in petascope.properties". Petascope endpoint (on port 9009) is in the ingredients.

But, you said "assume it's "internal" " then it is still like what I wrote in the ticket description.

"Assume it's internal" AND print a warning that it's making this assumption.. the quote from me below also says to print a warning, so I'm not sure what are you trying to "correct me".

When the user fixes petascope.properties → there will be no warning anymore.

if secore url is localhost (regardless of what port) then assume it's "internal" + print a warning to fix it in petascope.properties?

comment:4 by Bang Pham Huu, 3 years ago

Description: modified (diff)

comment:5 by Bang Pham Huu, 3 years ago

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.