@wangi commented on this pull request.
> @@ -68,7 +68,7 @@ export default function (map) {
$("#maxlat").val(ne.lat);
$("#export_overpass").attr("href",
-
`https://overpass-api.de/api/map?bbox=${sw.lng},${sw.lat},${ne.lng},${ne.lat}`);
+ `${OSM.OVERPASS_URL.replace(/\/interpreter$/,
"/map")}?bbox=${sw.lng},${sw.lat},${ne.lng},${ne.lat}`);
I was trying to keep it small and surgical. The benefit of leaving
`overpass_api` as it is is commonality - it's how the various servers out there
are normally listed.
>From experience, I know that with appropriate apache/nginx configuration you
>could set it up without the `api` prefix. I run optimised with
>Supervisord/FCGI, but even with the basic setup per
>https://overpass-api.de/full_installation.html it's just a change to
>`ScriptAlias`. However, not sure why anyone would.
If wanting to make it more generic then I'd recommend keeping the `/api` part
in the setting.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/7208#discussion_r3523745322
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/7208/review/[email protected]>_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
