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

Clarify Privacy Policy about web API #448

Merged
merged 1 commit into from
Nov 23, 2023
Merged

Clarify Privacy Policy about web API #448

merged 1 commit into from
Nov 23, 2023

Conversation

yvanzo
Copy link
Contributor

@yvanzo yvanzo commented Nov 20, 2023

Two small updates to our Privacy Policy:

  1. It explicitly mentions web API calls in the paragraph about server logs. It comes after a user’s question on IRC.
  2. It drops FTP as we don’t use it anymore, datasets now have to be downloaded from web servers through HTTPS even though ftp remains in a couple of subdomain names for now.

@yvanzo yvanzo requested review from mayhem and zas November 20, 2023 08:10
Copy link

github-actions bot commented Nov 20, 2023

Test Results

67 tests  ±0   67 ✔️ ±0   8s ⏱️ ±0s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ±0 

Results for commit 6d34d08. ± Comparison against base commit ff0a78b.

♻️ This comment has been updated with latest results.

It explicitly mentions web API calls in the paragraph about server logs.
@yvanzo yvanzo changed the title Clarify Privacy Policy about web API and drop FTP Clarify Privacy Policy about web API Nov 23, 2023
@yvanzo
Copy link
Contributor Author

yvanzo commented Nov 23, 2023

I reverted dropping FTP as it is still up and running even though not listed from our download pages.

Copy link

@zas zas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yvanzo yvanzo merged commit 1c456b0 into master Nov 23, 2023
2 checks passed
@yvanzo yvanzo deleted the api-policy branch November 23, 2023 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants