georchestra / mapstore2-georchestra

geOrchestra newest viewer
Other
6 stars 23 forks source link

Change proxy url path to root #604

Open tkohr opened 1 year ago

tkohr commented 1 year ago

This PR changes proxy requests to be sent from the root path https://[mygeorchestra]/ instead of https://[mygeorchestra]/mapstore/

When trying to use an external layer (3dtiles in the occasion) via the proxy I stumbled over the fact that requests to the layer sent from https://[mygeorchestra]/mapstore/ are blocked by the georchestra security proxy.

Wondering if anybody has already successfully used the proxy with the current url within georchestra. If so, please correct me, I might be missing something.

landryb commented 1 year ago

mapstore itself uses its own proxy at /mapstore/proxy (hence the missing leading slash?) which is different from georchestra's internal proxy at /proxy. Iirc it's supposed to work too, and be used for urls not in useCORS list...

tkohr commented 1 year ago

Ok, I didn't get the requests via the mapstore proxy ("url": "proxy/?url=") to work then (maybe due to missing configuration?). The same requests worked via the georchestra proxy ("url": "/proxy/?url="). I'm testing on https://github.com/georchestra/docker/tree/22.0

As the server I'm querying accepts CORS I finally don't use a proxy anymore.

Just opened this PR, in case anybody else stumbles on same problem and can confirm.