Cadoles/goweb-oidc
10
0
Fork 0
Code Issues 1 Pull Requests Releases 4 Wiki Activity

Injection de la redirect_uri dans les requêtes d'authentification #5

Merged
lgourvenec merged 2 commits from issue-4 into master 2025-03-11 15:28:41 +01:00
Conversation 2 Commits 2 Files Changed 14 +298 -16
wpetit commented 2025-03-07 12:09:06 +01:00
Owner
Copy Link

Description

Cette pull request vise à résoudre un problème lié à la gestion des redirections après authentification OpenID Connect (OIDC) dans le projet Cadoles/goweb-oidc. L'objectif est de permettre la spécification de la redirect_uri dans l'URL de redirection lors d'un appel à /login, notamment dans les cas où un client OIDC partagé entre plusieurs applications présente plusieurs redirectUris. Cela devrait résoudre les erreurs de redirect_uri renvoyées par Hydra.

Changements

  • Mise à jour du fichier cmd/server/container.go pour inclure la possibilité de spécifier la redirect_uri (OIDC_REDIRECT_URL) dans l'URL de redirection lors de l'appel à /login.
  • Modification de la configuration de Docker Compose dans docker-compose.yml pour refléter les changements nécessaires à la gestion des redirections et à l'intégration avec Hydra.
  • Ajout de nouveaux fichiers et modifications dans le répertoire misc/compose/hydra pour supporter la configuration d'Hydra et la gestion des clients OIDC.
  • Mise à jour des scripts dans misc/compose/hydra pour gérer les clients OIDC et les redirections de manière appropriée.
  • Modification du Dockerfile dans misc/docker pour utiliser l'image Go 1.24 au lieu de 1.19 pour la construction de l'image Docker.
  • Ajouts et modifications divers dans les fichiers de configuration et les scripts pour supporter les nouvelles fonctionnalités et résoudre les problèmes de redirection.

Tester

L'image reg.cadoles.com/wpetit/oidc-tester:2025.3.7-issue4.1208.e2e4915 est disponible pour tester cette évolution.

## Description Cette pull request vise à résoudre un problème lié à la gestion des redirections après authentification OpenID Connect (OIDC) dans le projet Cadoles/goweb-oidc. L'objectif est de permettre la spécification de la `redirect_uri` dans l'URL de redirection lors d'un appel à `/login`, notamment dans les cas où un client OIDC partagé entre plusieurs applications présente plusieurs `redirectUris`. Cela devrait résoudre les erreurs de `redirect_uri` renvoyées par Hydra. ## Changements - Mise à jour du fichier `cmd/server/container.go` pour inclure la possibilité de spécifier la `redirect_uri` (`OIDC_REDIRECT_URL`) dans l'URL de redirection lors de l'appel à `/login`. - Modification de la configuration de Docker Compose dans `docker-compose.yml` pour refléter les changements nécessaires à la gestion des redirections et à l'intégration avec Hydra. - Ajout de nouveaux fichiers et modifications dans le répertoire `misc/compose/hydra` pour supporter la configuration d'Hydra et la gestion des clients OIDC. - Mise à jour des scripts dans `misc/compose/hydra` pour gérer les clients OIDC et les redirections de manière appropriée. - Modification du `Dockerfile` dans `misc/docker` pour utiliser l'image Go 1.24 au lieu de 1.19 pour la construction de l'image Docker. - Ajouts et modifications divers dans les fichiers de configuration et les scripts pour supporter les nouvelles fonctionnalités et résoudre les problèmes de redirection. ## Tester L'image `reg.cadoles.com/wpetit/oidc-tester:2025.3.7-issue4.1208.e2e4915` est disponible pour tester cette évolution.
wpetit added 2 commits 2025-03-07 12:09:07 +01:00
fix: isolated local dev environment
Some checks reported warnings
Cadoles/goweb-oidc/pipeline/head This commit was not built
Details
97bacd0bb8
fix: inject redirect_uri in oauth2 login request (#4)
All checks were successful
Cadoles/goweb-oidc/pipeline/head This commit looks good
Details
Cadoles/goweb-oidc/pipeline/pr-master This commit looks good
Details
e2e4915e93
wpetit changed title from issue-4 to Injection de la `redirect_uris` dans les requêtes d'authentification 2025-03-07 12:10:53 +01:00
jenkins commented 2025-03-07 12:11:05 +01:00
Owner
Copy Link

Test report for e2e4915

Output 
go test -v -race ./...
?   	forge.cadoles.com/cadoles/goweb-oidc	[no test files]
?   	forge.cadoles.com/cadoles/goweb-oidc/cmd/server	[no test files]
?   	forge.cadoles.com/cadoles/goweb-oidc/internal/config	[no test files]
?   	forge.cadoles.com/cadoles/goweb-oidc/internal/route	[no test files]
# Test report for e2e4915 <details open> <summary>Output</summary> ``` go test -v -race ./... ? forge.cadoles.com/cadoles/goweb-oidc [no test files] ? forge.cadoles.com/cadoles/goweb-oidc/cmd/server [no test files] ? forge.cadoles.com/cadoles/goweb-oidc/internal/config [no test files] ? forge.cadoles.com/cadoles/goweb-oidc/internal/route [no test files] ``` </details>
wpetit changed title from Injection de la `redirect_uris` dans les requêtes d'authentification to Injection de la `redirect_uri` dans les requêtes d'authentification 2025-03-07 12:18:52 +01:00
lgourvenec approved these changes 2025-03-11 15:28:30 +01:00
lgourvenec left a comment
Owner
Copy Link

Testé avec un cluster kind -> problème résolu.

Testé avec un cluster kind -> problème résolu.
lgourvenec merged commit 6545cb3e93 into master 2025-03-11 15:28:41 +01:00
lgourvenec deleted branch issue-4 2025-03-11 15:28:43 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
lgourvenec
Labels
No items
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Cadoles/goweb-oidc#5
No description provided.
Delete Branch "issue-4"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?