IDR / idr-gallery

https://pypi.org/project/idr-gallery/
GNU Affero General Public License v3.0
1 stars 1 forks source link

mapr redirects not working on idr-next #21

Closed will-moore closed 1 year ago

will-moore commented 1 year ago

See https://github.com/IDR/deployment/pull/388

For the mapr redirect logic, we need to look-up Map-Ann Keys for a given value, added in https://github.com/IDR/idr-gallery/pull/15

This works OK on idr-testing and idr production, but not on idr-next, since the idr-next searchengine is not available from where the Django code is running.

sbesson commented 1 year ago

My understanding is that the code introduced in #15 resolves the base URL passed by the initial request call sent by the client and uses it to make a secondary request call within the deployment itself to retrieve the content of another deployment.

Currently idr-next.openmicroscopy.org is not a public DNS and even the idr-next deployments do not know about it. So any internal request to https://idr-next.openmicroscopy.org/searchengine/api/v1/resources/ will fail with a resolution error e.g.

[sbesson@prod110-proxy ~]$ curl https://idr-next.openmicroscopy.org/searchengine/api/v1/resources
curl: (6) Could not resolve host: idr-next.openmicroscopy.org; Unknown error

A few immediate options:

/cc @francesw @jburel

will-moore commented 1 year ago

Using JS to do this would not give such a nice redirect experience. You'd have to load a page with JS, the JS would make a query and then redirect -> reload a 2nd page.

I can try relative URLs instead of the absolute URLs given by Django...

will-moore commented 1 year ago

Closing - I think we can accept that re-directs won't work on idr-next. Doesn't break other testing.