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

mime-type n-triples isn't compressed #472

Closed
Rdataflow opened this issue Aug 21, 2024 · 3 comments · Fixed by #473
Closed

mime-type n-triples isn't compressed #472

Rdataflow opened this issue Aug 21, 2024 · 3 comments · Fixed by #473

Comments

@Rdataflow
Copy link
Contributor

repro steps

curl -v 'https://test.ld.admin.ch/query' -X POST -H 'Accept: application/n-quads' -H 'Accept-Language: de,en-US;q=0.7,en;q=0.3' -H 'Accept-Encoding: gzip, deflate, br, zstd' -H 'Content-Type: application/x-www-form-urlencoded' --data-raw 'query=CONSTRUCT%20%7B%20%3Fsub%20%3Fpred%20%3Fobj%20.%20%7D%20%0AWHERE%20%7B%0A%20%20%3Fsub%20%3Fpred%20%3Fobj%20.%0A%7D%20LIMIT%201000' > /dev/null

expected
response header with content-encoding: gzip

@ludovicm67
Copy link
Member

@Rdataflow Thank you for the bug report and the PR. I will take a look 👍

@Rdataflow
Copy link
Contributor Author

I just went through all of the YASGUI dropdown format options for a SELECT and a CONSTRUCT to check if something doesn't compress. Feel free to add or alter whatever seems best to you 👍

@ludovicm67
Copy link
Member

From all the test I did, the application/sparql-results+json; charset=utf-8 was not able to work anymore with your changes. I added a commit to add support for parameterized content-types to solve the issue 😄

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