Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No compression on osm.org "Map Notes" layer #1089

Closed
mmd-osm opened this issue May 31, 2024 · 5 comments
Closed

No compression on osm.org "Map Notes" layer #1089

mmd-osm opened this issue May 31, 2024 · 5 comments

Comments

@mmd-osm
Copy link

mmd-osm commented May 31, 2024

Although my browser has requested the "Map Notes" layer with compression, the response appears to be uncompressed.

I believe this could be somehow related to the ".json" suffix in the URL below. Either removing ".json", or replacing it by ".xml" enables compression again. Since this might be some Apache setting, I've opened the issue on the operations repo.

Sample request:

curl 'https://www.openstreetmap.org/api/0.6/notes.json?bbox=17.354106903076175,52.47713460532417,17.810554504394535,52.556316065406556'  -H 'Accept: */*' -H 'Accept-Language: en' -H 'Accept-Encoding: gzip, deflate, br, zstd'
@tomhughes
Copy link
Member

You haven't requested it with compression, you have said you are able to accept compression if the server so chooses. Apparently it chose not to in this case.

In any case I don't believe this is an operational matter as I don't believe we use apache on the fly compression at all.

@tomhughes
Copy link
Member

Actually maybe we do, but you confused me with the suggestion that this was specific to the .json extension but it isn't as it happens for any JSON response regardless of extension.

@tomhughes tomhughes reopened this May 31, 2024
@tomhughes
Copy link
Member

Yes you can get JSON with /notes if you set Accept: application/json in the request - you don't need the extension.

@tomhughes
Copy link
Member

The other thing was that it's part of our generic apache configuration so I couldn't see it in the web site config at all.

Now fixed in openstreetmap/chef@49e19a7 anyway.

@mmd-osm
Copy link
Author

mmd-osm commented May 31, 2024

Apologies, there's been a bit of an overlap of my answer with your update. Thank you for the quick fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants