Telemetry: a persistent machine_id makes the data pseudonymous, and default-on collection lacks prior consent for EU users (GDPR / ePrivacy)
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- markdown, sql
- Área
- backend, documentation, security
Línea de trabajo
Start by reading TELEMETRY.md, the telemetry command and CLI notice, and telemetry-worker/migrations/0001_init.sql; trace where machine_id is created, stored, sent, and retained. Then compare the suggested opt-in, identifier rotation or removal, retention, and wording changes. Done means the project has chosen and implemented a coherent approach, with documentation and user-facing messages matching its behavior.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Summary
Telemetry is described as anonymous, but every payload carries machine_id, a random UUID stored in ~/.codegraph/telemetry.json that never expires. Because it links all events from the same machine over time, the data is pseudonymous rather than anonymous. Under the GDPR, pseudonymous data is still personal data. Combined with the default-on setting, I believe the current design is unlikely to be compliant for users in the EU, and I'd like to suggest some changes.
Context: CodeGraph 1.6.0 on Windows 11 (npm install). I never ran the interactive installer, so no toggle was shown. The one-line notice appeared during my first codegraph index, in the same run as the first POST, so there was no opportunity to refuse before data left the machine.
Details
-
Pseudonymous, not anonymous. A stable identifier that lets you recognise the same machine across sends is pseudonymisation (GDPR Art. 4(5), Recital 26), and the data stays within the scope of the GDPR. The words "anonymous usage stats" in TELEMETRY.md, the CLI notice and the
telemetrycommand output are therefore misleading for EU users. The client IP is also exposed to the hosting provider when the request is made, even though the worker itself never reads it. -
Consent before storing an identifier on the device. Writing an identifier to the user's machine and reading it back for analytics falls under Art. 5(3) of the ePrivacy Directive (in France, Article 82 of the Loi Informatique et Libertés). It requires prior consent unless the storage is strictly necessary for the service the user asked for, and usage statistics are not. The EDPB Guidelines 2/2023 on the technical scope of Art. 5(3) make clear this is not limited to browser cookies. A default-on toggle, or a notice printed when the first send happens, is not valid consent: consent must be a clear affirmative act (CJEU C-673/17, Planet49).
-
Retention. Raw events are purged after 90 days, but according to
telemetry-worker/migrations/0001_init.sql,machine_daysandmachine_first_seenkeep the machine ID and its active days with no limit. For comparison, the French regulator (CNIL) only exempts audience measurement from consent when the identifier lasts at most 13 months and the data is kept at most 25 months. The current design does not meet those conditions either.
Suggestions
- Make telemetry opt-in, or at least keep it off in non-interactive runs until the user has explicitly made a choice.
- If a persistent ID is kept, rotate it (for example every 13 months at most) and put a retention limit on
machine_daysandmachine_first_seen. Alternatively, drop the persistent ID in favour of a per-day random value. - Describe the data as pseudonymous in TELEMETRY.md, the notice and the CLI messages. Also name the data controller and give a contact for access and erasure requests (a deletion request can only be matched through the machine ID).
I'm not a lawyer, and I'm happy to be corrected on any of these points. Thanks for the detailed TELEMETRY.md and for publishing the worker code, which made all of this easy to check.
- Lenguaje dominante
- C
- Estrellas
- 73.2k
- Forks
- 4.7k
- Merge medio
- 7 h 49 min
- PR fusionados (30 d)
- 629
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de colbymchenry/codegraph
-
codegraph ui - does not workAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
colbymchenry/codegraph#1900 · 2 comentarios · 5 reacciones ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
colbymchenry/codegraph#1700 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
colbymchenry/codegraph#1535 · 2 comentarios · 2 reacciones ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 63/100
colbymchenry/codegraph#1403 ·
Los mantenedores suelen responder en 1 día
-
agent-target
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
colbymchenry/codegraph#649 · 5 comentarios · 50 reacciones ·
Los mantenedores suelen responder en 1 día
Todos los issues de colbymchenry/codegraph
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
zephyrproject-rtos/zephyr#123423 ·
Los mantenedores suelen responder en 2 días
-
dspromote: /etc/exports uses -alldirs on non-mount-point directories, so mountd exports nothingAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
nextbsd/nextbsd-userland#434 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
Los mantenedores suelen responder en 1 día
-
[YSQL] Local-node initdb reads postgres.bki, but YB clusters bootstrap only from yb_postgres.bkiAbiertoarea/ysql kind/enhancement priority/low status/awaiting-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
yugabyte/yugabyte-db#35763 ·
Los mantenedores suelen responder en 1 día
-
[mimalloc] update to 3.6.0Abiertocategory:port-update
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
Los mantenedores suelen responder en 2 días