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.

Sign up for GitHub

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

Jump to bottom

query: Unescaped CR/LF in GeoJSON output #93

Closed
clarisma opened this issue Feb 12, 2023 · 1 comment
Closed

query: Unescaped CR/LF in GeoJSON output #93

clarisma opened this issue Feb 12, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@clarisma
Copy link
Owner

clarisma commented Feb 12, 2023 •
edited

GeoJSON output performs simplified string escaping, as it relies on the following:

OSM strings have already been cleaned up during import

But this is not true, only strings that are written into string tables are cleaned this way; rare strings are not cleaned.

Should spec be changed? See #94
@clarisma clarisma added the bug Something isn't working label Feb 12, 2023
clarisma added a commit that referenced this issue Aug 26, 2023
@clarisma
Proper escaping for GeoJSON
c2af518 
Fixes #101, #93
@clarisma
Copy link
Owner Author

Fixed in c2af518

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@clarisma