chore: Made oauth2 scopes required in CLI (#3165)

This commit is contained in:
CEbbinghaus 2024-11-01 12:59:27 +11:00 committed by GitHub
parent cc7530aa65
commit 1b58e4169a
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 7 additions and 3 deletions

View file

@ -210,7 +210,7 @@ You can create a scope map with:
```bash ```bash
kanidm system oauth2 update-scope-map <name> <kanidm_group_name> [scopes]... kanidm system oauth2 update-scope-map <name> <kanidm_group_name> [scopes]...
kanidm system oauth2 update-scope-map nextcloud nextcloud_admins admin kanidm system oauth2 update-scope-map nextcloud nextcloud_users email profile openid
``` ```
> [!TIP] > [!TIP]
@ -225,13 +225,17 @@ kanidm system oauth2 update-scope-map nextcloud nextcloud_admins admin
> - **email** - email, email_verified > - **email** - email, email_verified
> - **address** - address > - **address** - address
> - **phone** - phone_number, phone_number_verified > - **phone** - phone_number, phone_number_verified
> - **groups** - groups
<!-- this is just to split the templates up --> <!-- this is just to split the templates up -->
> [!WARNING] > [!WARNING]
> >
> If you are creating an OpenID Connect (OIDC) client you **MUST** provide a scope map named > If you are creating an OpenID Connect (OIDC) client you **MUST** provide a scope map containing
> `openid`. Without this, OpenID Connect clients **WILL NOT WORK**! > `openid`. Without this, OpenID Connect clients **WILL NOT WORK**!
> ```bash
> kanidm system oauth2 update-scope-map nextcloud nextcloud_users openid
> ```
You can create a supplemental scope map with: You can create a supplemental scope map with:

View file

@ -943,7 +943,7 @@ pub struct Oauth2CreateScopeMapOpt {
nopt: Named, nopt: Named,
#[clap(name = "group")] #[clap(name = "group")]
group: String, group: String,
#[clap(name = "scopes")] #[clap(name = "scopes", required = true)]
scopes: Vec<String>, scopes: Vec<String>,
} }