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

Privacy Policy reworked #35

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

comradekingu
Copy link
Contributor

No description provided.

@ndfred
Copy link
Owner

ndfred commented Jun 30, 2020

Thanks for taking a stab at this (and all the other docs)! Since this is legalese, and I got it from Overcast, I would rather not change it without understanding the implications here.


We do not collect any data through the app, the only data that is exchanged is between the app and the iPerf 3 server you are connecting to. The help section is embedded in the app and does not connect to a server. We recommend [running your own iPerf 3 server](https://github.com/ndfred/iperf-ios/blob/master/Documentation/Help.md).
No data is collected through the app. Data is only exchanged between the app and the iPerf 3 server it connects to. The help section is embedded in the app and does not connect to a server. It is recommended to [run your own iPerf 3 server](https://github.com/ndfred/iperf-ios/blob/master/Documentation/Help.md).
Copy link
Contributor Author

@comradekingu comradekingu Jul 1, 2020

Choose a reason for hiding this comment

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

Who is "We", and what amounts to "data". This sentence is bad.
Why is it recommended to run your own server?

* finally the [test results](https://github.com/ndfred/iperf-ios/blob/master/Source/iperf3/iperf_api.c#L1690) are sent to the server
* The app connects to an iPerf 3 server and sends [the test's parameters](https://github.com/ndfred/iperf-ios/blob/master/Source/iperf3/iperf_api.c#L1526).
* Random test data is sent to / and from the server.
* Finally the [test results](https://github.com/ndfred/iperf-ios/blob/master/Source/iperf3/iperf_api.c#L1690) are sent to the server.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Using what? Protocol, encryption, etc.


## Analytics

The app does not collect any analytics beyond what is needed to run an iPerf 3 test as described in the prior section.
The app itself does not collect any info, and its operation does not exchange any info with the server that is considered personal data as per [article 4 of the EU-GDPR](https://gdpr-info.eu/art-4-gdpr/). The app does not collect or send anything beyond what is described in "Collected info".
Copy link
Contributor Author

Choose a reason for hiding this comment

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

There are no analytics. What consititutes "the app" is varies distinctly based on where it is fetched from. While not incorrect, in the interest of not being spied on, I tried to make this point clearer. Still not good.


## Your Consent

By using this app, you consent to our privacy policy.
By using this app, you consent to exchange info as described above with the server you are using it with.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Collecting consent this way is legally null and void. Explaining that it can't do anything if this doesnt happen, as that is what it does, is relevant.


If we decide to change our privacy policy, we will post those changes on this page. Summary of changes so far:
Changes to our privacy policy so far:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should be a link to the full version.

@comradekingu
Copy link
Contributor Author

That makes sense. I was wondering why it happened to be so strangely made.
I commented myself through the changes. Will have another go at it. But first, what do you think about my comments?

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