-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat: Authentification obligatoire sur l'API #1628
base: master
Are you sure you want to change the base?
Conversation
a62a34a
to
a64a402
Compare
2e7a607
to
7a1afc9
Compare
7a1afc9
to
d8d087e
Compare
d8d087e
to
507e446
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Juste quelques remarques sur les tests. Il faut ajuster le nom des tests quand on prend désormais une 401 et il faut continuer de voir si on a bien les données quand on est connecté.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LG !
815a0a0
to
567e23c
Compare
️✅ There are no secrets present in this pull request anymore.If these secrets were true positive and are still valid, we highly recommend you to revoke them. 🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request. |
a83cb45
to
7fc2bef
Compare
3399ad3
to
0084d1c
Compare
0084d1c
to
c2a3fdb
Compare
Quoi ?
Ne plus autoriser les utilisateur à accéder librement à l'API
Pourquoi ?
Les utilisateurs se connectent trop en anonyme, on ne peut pas faire de stats dessus
Comment ?