Skip to content

RGPD: Ne pas mettre les adresses géolocalisées par les management commands dans la sortie standard - #5583

Merged
tonial merged 1 commit into
masterfrom
alaurent/codeql
Feb 10, 2025
Merged

RGPD: Ne pas mettre les adresses géolocalisées par les management commands dans la sortie standard#5583
tonial merged 1 commit into
masterfrom
alaurent/codeql

Conversation

@tonial

@tonial tonial commented Feb 10, 2025

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Suite à cette remontée: https://github.com/gip-inclusion/les-emplois/security/code-scanning/129

Seules ces 2 commandes utilise ce code, et effectivement c'est mieux que ça ne finisse pas en print dans les données collectées par clever.

Je le rend facultatif pour permettre d'afficher les données en local lors de tests manuels (qui on a priori permis à victor de faire ces stats à l'époque).

Je me demande si on ne peut pas désormais supprimer update_companies_coords maintenant que c'est fait automatiquement à chaque modification dans l'admin 🤔

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?
  • Ajouter l'étiquette « Bug » ?

🏝️ Comment tester ?

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

@tonial tonial added the modifié Modifié dans le changelog. label Feb 10, 2025
@tonial tonial self-assigned this Feb 10, 2025
Comment thread itou/companies/management/commands/update_companies_coords.py Outdated

@xavfernandez xavfernandez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Après il faut avoir en tête que ces deux commandes ne sont jamais lancées par des crons et leurs sorties n’atterrissent donc jamais dans les logs.
Sinon, il faudrait remplacer quelques self.stdout.write et print par des logger.info 😛

Comment thread itou/companies/management/commands/update_companies_coords.py Outdated
@tonial

tonial commented Feb 10, 2025

Copy link
Copy Markdown
Contributor Author

Comme il est évoqué dans les commandes qu'on pourrait le faire régulièrement, j'ai préféré anticiper un potentiel futur problème

@tonial
tonial added this pull request to the merge queue Feb 10, 2025
Merged via the queue into master with commit 26220bc Feb 10, 2025
@tonial
tonial deleted the alaurent/codeql branch February 10, 2025 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants