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

Remove link to ndjson.org as it's now hosting malware #9107

Merged
merged 2 commits into from
Jan 19, 2024

Conversation

julian94
Copy link
Contributor

Removed link to ndjson.org as the author of it has abandoned the domain and it's now being used for hosting malware.

What does this PR do?

Removes the link to ndjson.org

What are related issues/pull requests?

This issue in the ndjson repo, explains that the domain has been abandoned.

Tasklist

  • Make sure code is correctly formatted (cf pre-commit configuration)
  • Add test case(s)
  • Add documentation
  • Updated Python API documentation (swig/include/python/docs/)
  • Review
  • Adjust for comments
  • All CI builds and checks have passed

Removed link to ndjson.org as the author of it has abandoned the domain and it's now being used for hosting malware.
@rouault
Copy link
Member

rouault commented Jan 19, 2024

could be worth pointing to https://en.wikipedia.org/wiki/JSON_streaming instead

@julian94
Copy link
Contributor Author

Yeah that link is a pretty good description. I'll add it.

@rouault rouault merged commit 070e391 into OSGeo:master Jan 19, 2024
3 checks passed
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

Successfully merging this pull request may close these issues.

2 participants