Repository navigation
Conversation
- les identifiants OAuth sont lus au démarrage (NUXT_*) et plus à la construction de l'image - docker-compose.yml transmet DISCORD_* et TWITCH_* à l'application - l'échange du code OAuth se fait côté serveur (/api/oauth/discord, /api/oauth/twitch) avec un state vérifié - un pseudo Discord ou Twitch n'ouvre jamais une session administrateur - coquille TWTICH_CLIENT_SECRET corrigée - tests locaux et Docker avec un faux serveur OAuth (tests/) - README et .env.example mis à jour Closes #182
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #182
Problème
Depuis #158,
.envn'est plus copié dans l'image Docker.nuxt.config.tslisaitDISCORD_*etTWITCH_*à la construction de l'image : les valeurs étaient vides, et le formulaire de connexion envoyait des identifiants vides à Discord et Twitch. La stack Komodo n'a de toute façon queJWT_SECRET.Ce que change cette PR
NUXT_*que Nuxt surcharge à l'exécution.docker-compose.ymlrecopieDISCORD_*etTWITCH_*vers ces variables./api/oauth/discord,/api/oauth/twitch) : le secret n'est plus lu dans le navigateur, et lestateest aléatoire et vérifié au retour.TWTICH_CLIENT_SECRETcorrigée, route/api/third-party/[user]supprimée.mainaprès Corrige les contrôles d'accès de l'API et les migrations en production #184, fix: compte administrateur, seed haché et API sans hash (#189) #190 et Passage à Nuxt 4 #188 : la vérification du mot de passe delogin.post.tsreste celle de Corrige les contrôles d'accès de l'API et les migrations en production #184..env.examplecomplété, README mis à jour (variables, Komodo, tests).À faire côté serveur pour que ça marche
Ajouter dans l'Environment de la stack Komodo :
DISCORD_CLIENT_ID,DISCORD_CLIENT_SECRET,DISCORD_CLIENT_REDIRECT,TWITCH_CLIENT_ID,TWITCH_CLIENT_SECRET,TWITCH_CLIENT_REDIRECT(voir.env.example), puis redéployer. Les URL de redirection doivent être déclarées à l'identique dans les applications Discord et Twitch.Tests
Aucun appel aux vrais services :
tests/fake-oauth.mjssimule Discord et Twitch.bash tests/run-local.sh: 18/18 (Node 20, build Nuxt 4).bash tests/run-docker.sh: 18/18 avec l'image Docker.prisma/milestone.dbn'est jamais touchée (copie dans un dossier temporaire).