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

[FEATURE] Autoriser l'accès à Pix Junior seulement si une session est active #9456

Conversation

aurelie-crouillebois
Copy link
Contributor

🦄 Problème

Pix Junior est accessible sans authentification et donne accès à la liste des élèves, importés dans une école, à partir d'un simple code école.
Ce code école peut être attaqué en force brute dans l'application.

🤖 Proposition

Pour limiter la surface d'attaque de ce code, celui-ci ne sera rendu utilisable qu'à condition qu'une session ait été ouverte par l'enseignant.
Cette PR a pour objectif d'empêcher l'accès à Pix Junior lorsqu'il n'y a pas de session active pour l'école.

🌈 Remarques

L'API doit se comporter de la même manière lorsqu'on lui fournit un code école erroné, ou un code école où aucune session n'est ouverte. Ceci pour ne pas donner d'information sur la validité du code école.

💯 Pour tester

Sur la page de saisie du code école de Pix Junior :

  • saisir un code inconnu : la page indiquant que le code école est erroné ou que la session de l'école est fermée.
  • saisir un code école pour une école valide (sans session ouverte) : idem
  • ouvrir une session et saisir le code école : la liste des classes s'affiche
  • attendre la fermeture de session (ou aller changer la date de fin en bdd) : la page d'école non trouvée s'affiche de nouveau

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@aurelie-crouillebois aurelie-crouillebois force-pushed the pix-12727-add-session-verification-for-all-PixJunior-route branch from 4c6b77c to f0c1377 Compare July 5, 2024 12:30
@yaelle6 yaelle6 marked this pull request as ready for review July 8, 2024 08:01
@yaelle6 yaelle6 requested a review from a team as a code owner July 8, 2024 08:01
@yaelle6 yaelle6 changed the title [FEATURE] Autoriser l'accès à Pix Junior seulement si un session est active [FEATURE] Autoriser l'accès à Pix Junior seulement si une session est active Jul 8, 2024
@rouxxi rouxxi force-pushed the pix-12727-add-session-verification-for-all-PixJunior-route branch 2 times, most recently from 442abdb to 86822ae Compare July 8, 2024 09:05
@yaelle6 yaelle6 force-pushed the pix-12727-add-session-verification-for-all-PixJunior-route branch from 86822ae to 3967d93 Compare July 8, 2024 09:31
Copy link
Contributor

@yaelle6 yaelle6 left a comment

Choose a reason for hiding this comment

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

review tech et front en mob

@yaelle6 yaelle6 force-pushed the pix-12727-add-session-verification-for-all-PixJunior-route branch from 3967d93 to 025ef04 Compare July 8, 2024 10:26
@pix-service-auto-merge pix-service-auto-merge merged commit d05867f into dev Jul 8, 2024
10 of 12 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-12727-add-session-verification-for-all-PixJunior-route branch July 8, 2024 12:10
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.

None yet

7 participants