Issue 555: Skyline assumes /labkey context path when pinging Panorama

Status:closed
Assigned To:Guest
Type:Defect
Area:Skyline
Priority:3
Milestone:4.1
Opened:2017-10-24 19:33 by Josh Eckels
Changed:2018-10-03 09:44 by Vagisha Sharma
Resolved:2018-10-03 09:43 by Vagisha Sharma
Resolution:Fixed
Closed:2018-10-03 09:44 by Vagisha Sharma
2017-10-24 19:33 Josh Eckels
Title»Skyline assumes /labkey context path when pinging Panorama
Assigned ToGuest»Vagisha
Type»Defect
Area»Skyline
Priority»3
Milestone»4.1
I ran into this today debugging a problem for a Panorama Partner. They couldn't get their Skyline instance to accept a valid Panorama server URL. They were getting a dialog with a message along the lines of "The server X is not a Panorama server"

The issue turned out to be that Tomcat had LabKey deployed to the root context path, meaning that the URLs didn't start with "/labkey"

There are at least a couple of places in PanoramaUtils.cs that assume /labkey-based URL, including:

/labkey/security/home/ensureLogin.view

/labkey/project/home/getContainers.view

I propose that we switch Skyline so that if you don't specify a context path when configuring a Panorama server URL, we try both the root and /labkey before giving up. We should also accept URLs that use other context paths for newly configured servers.

2018-10-03 09:43 Vagisha Sharma
resolve as Fixed
Statusopen»resolved
Fixed in:
SHA-1: ea78ec3fb12d23c60fcbf17c1a94e0ccb69bda98

* Skyline:
Merged revision(s) 11637 from branches/skyline_3_7:
Skyline: Support context paths other than 'labkey' and handle panoramaweb redirects on ensureLogin.

2018-10-03 09:44 Vagisha Sharma
close
Statusresolved»closed
Assigned ToVagisha»Guest