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

Add option to disable app telemetry #52

Open
thomasnordquist opened this issue Jan 27, 2019 · 9 comments
Open

Add option to disable app telemetry #52

thomasnordquist opened this issue Jan 27, 2019 · 9 comments

Comments

@thomasnordquist
Copy link
Owner

@thomasnordquist thomasnordquist commented Jan 27, 2019

App telemetry is used to automatically detect bugs and measure acceptance.
One should be able to disable the telemetry.

See:
https://github.com/thomasnordquist/MQTT-Explorer#telemetry

@thomasnordquist
Copy link
Owner Author

@thomasnordquist thomasnordquist commented Jul 17, 2019

Moved example telemetry from readme:

{ system: { arch: 'x64', platform: 'darwin' },
  appVersion: '0.0.7',
  events: { HELLO_EVENT: [ 1547714886134 ] },
  now: 1547714886135,
  transactionId: '1767d251-f492-4f2c-aa62-88add3acc26b' }
{ errors:
   [ { time: 1547714887921,
       message: 'He\'s dead Jim!',
       stack:
        'Error: He\'s dead Jim!\n    at ./src/tracking.ts.exports.default (./mqtt-explorer/app/build/bundle.js:142765:11)\n    at new Promise (<anonymous>)\n    at Object../src/tracking.ts (./mqtt-explorer/app/build/bundle.js:142764:1)\n    at __webpack_require__ (./mqtt-explorer/app/build/bundle.js:20:30)\n    at Object../src/index.tsx (./mqtt-explorer/app/build/bundle.js:142618:1)\n    at __webpack_require__ (./mqtt-explorer/app/build/bundle.js:20:30)\n    at ../backend/node_modules/charenc/charenc.js.charenc.utf8.stringToBytes (./mqtt-explorer/app/build/bundle.js:84:18)\n    at ./mqtt-explorer/app/build/bundle.js:87:10' } ],
  now: 1547714887921,
  transactionId: '53bf9aac-e695-40cc-9a81-b1cf3398843d' }
thomasnordquist added a commit that referenced this issue Jul 22, 2019
@GregoryEAllen

This comment was marked as off-topic.

@thomasnordquist

This comment was marked as off-topic.

@GregoryEAllen

This comment was marked as off-topic.

@thomasnordquist

This comment was marked as off-topic.

@GregoryEAllen

This comment was marked as off-topic.

@thomasnordquist
Copy link
Owner Author

@thomasnordquist thomasnordquist commented Jul 24, 2019

@GregoryEAllen no worries, I'm always happy to get error reports. It helps tremendously maintaining good software quality.

I just remove the whole conversation in a few minutes 😇

@pintman
Copy link

@pintman pintman commented Aug 16, 2019

It would be great to have telemetry turned off by default or at least ask the user which option to choose. Having telemetry turned on by default may cause a GDPR violation in europe. Maybe you can reconsider the default behavior.

@aniston
Copy link

@aniston aniston commented Nov 12, 2020

Hello @thomasnordquist , lovely APP, but as @pintman pointed out using Telemetry without users consent will go against the EU GDPR rules even if there is a clear mention that telemetry will be taken (although no mention if only in Error cases or even thru normal usage?).

So the question as the OP dated back end of Jan 2019, is this option implemented as of Nov 2020 ?
might be better to keep it off as default and let users turn it on

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

Successfully merging a pull request may close this issue.

None yet
4 participants