entrouvert
/
authentic
16
0
Fork 0
Code Issues Pull Requests 33 Releases Activity

idp_oidc: add client_id & client_secret fields in edit form (#76224) #243

Closed
yweber wants to merge 2 commits from wip/76224-enable-client-id-secret-edition-for-oidc-service into wip/76223-oidc-service-slug-edition
pull from: wip/76224-enable-client-id-secret-edition-for-oidc-service
merge into: entrouvert:wip/76223-oidc-service-slug-edition
Conversation 3 Commits 2 Files Changed 5 +96 -1
yweber commented 2024-01-25 17:53:38 +01:00
Owner
Copy Link
No description provided.
yweber force-pushed wip/76224-enable-client-id-secret-edition-for-oidc-service from 3ec9cfe406 to 6631378ab3 2024-01-25 18:04:27 +01:00 Compare
yweber force-pushed wip/76224-enable-client-id-secret-edition-for-oidc-service from 6631378ab3 to c0b86c852d 2024-01-25 18:10:34 +01:00 Compare
yweber reviewed 2024-01-25 18:12:07 +01:00
tests/idp_oidc/test_manager.py Outdated
@ -217,0 +221,4 @@
resp = app.get(f'/manage/services/{oidc_client.id}/settings/edit/')
form = resp.form
form['client_id'] = 'superid'
form['client_secret'] = 'hackme'
yweber commented 2024-01-25 18:12:07 +01:00
Author
Owner
Copy Link

Il faudrait peut être valider la valeur qui est passé en client_secret (client_id aussi ?) ? Comment on test la longueur ? On test que c'est bien un uuid ?

Il faudrait peut être valider la valeur qui est passé en client_secret (client_id aussi ?) ? Comment on test la longueur ? On test que c'est bien un uuid ?
yweber force-pushed wip/76224-enable-client-id-secret-edition-for-oidc-service from c0b86c852d to c012b1409f 2024-01-29 09:20:18 +01:00 Compare
yweber changed title from WIP: idp_oidc: add client_id & client_secret fields in edit form (#76224) to idp_oidc: add client_id & client_secret fields in edit form (#76224) 2024-01-29 09:24:33 +01:00
yweber closed this pull request 2024-01-30 15:35:04 +01:00
tnoel reviewed 2024-01-30 16:08:39 +01:00
tnoel left a comment
Owner
Copy Link

Un peu de mal à comprendre la complexité de la machinerie ici. On veut pas juste avoir, lors de l'affichage du formulaire de création, des champs pré-remplis (avec des valeurs aléatoires) ? Je ne pense pas que du javascript soit nécessaire... ou alors je rate un truc ?

Un peu de mal à comprendre la complexité de la machinerie ici. On veut pas juste avoir, lors de l'affichage du formulaire de création, des champs pré-remplis (avec des valeurs aléatoires) ? Je ne pense pas que du javascript soit nécessaire... ou alors je rate un truc ?
yweber commented 2024-01-30 16:11:45 +01:00
Author
Owner
Copy Link

Un peu de mal à comprendre la complexité de la machinerie ici. On veut pas juste avoir, lors de l'affichage du formulaire de création, des champs pré-remplis (avec des valeurs aléatoires) ? Je ne pense pas que du javascript soit nécessaire... ou alors je rate un truc ?

Ben c'est pas ce que j'avais compris. Je croyais qu'il fallait permettre de changer la valeur du client_secret, donc :

  • a l'édition affiché la valeur courante (pour ne pas la changer quand on valide le formulaire)
  • permettre de générer un nouveau secret

Mais j'ai peut être mal compris le ticket ?

Et désolé, je me suis raté sur les histoires de suppressions de branches avec ma PR, j'ai du recréer cette PR ici : #250

> Un peu de mal à comprendre la complexité de la machinerie ici. On veut pas juste avoir, lors de l'affichage du formulaire de création, des champs pré-remplis (avec des valeurs aléatoires) ? Je ne pense pas que du javascript soit nécessaire... ou alors je rate un truc ? Ben c'est pas ce que j'avais compris. Je croyais qu'il fallait permettre de changer la valeur du client_secret, donc : - a l'édition affiché la valeur courante (pour ne pas la changer quand on valide le formulaire) - permettre de générer un nouveau secret Mais j'ai peut être mal compris le ticket ? Et désolé, je me suis raté sur les histoires de suppressions de branches avec ma PR, j'ai du recréer cette PR ici : https://git.entrouvert.org/entrouvert/authentic/pulls/250
fpeters commented 2024-01-30 16:23:58 +01:00
Owner
Copy Link

Mais j'ai peut être mal compris le ticket ?

Pour être clair, c'était dans la description ceci : "avec peut-être pour client_secret du code pour la génération automatique d'un nouveau secret". Dison qu'on peut zapper ça, ça sera plus simple.

> Mais j'ai peut être mal compris le ticket ? Pour être clair, c'était dans la description ceci : "avec peut-être pour client_secret du code pour la génération automatique d'un nouveau secret". Dison qu'on peut zapper ça, ça sera plus simple.
yweber commented 2024-01-30 17:07:09 +01:00
Author
Owner
Copy Link

Encore navré pour mon cafouillage de merge/suppression de branche/fermeture de PR...

Mais on ferait mieux d'échanger sur #250 sur laquelle j'ai aussi des commentaires de @bdauvergne

Encore navré pour mon cafouillage de merge/suppression de branche/fermeture de PR... Mais on ferait mieux d'échanger sur https://git.entrouvert.org/entrouvert/authentic/pulls/250 sur laquelle j'ai aussi des commentaires de @bdauvergne
All checks were successful
gitea/authentic/pipeline/head This commit looks good
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/authentic#243
No description provided.
Delete Branch "wip/76224-enable-client-id-secret-edition-for-oidc-service"

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?