]> git.proxmox.com Git - pve-access-control.git/commitdiff
fix #4074: increase API OpenID code size limit to 2048
authorMira Limbeck <m.limbeck@proxmox.com>
Wed, 15 Jun 2022 14:09:50 +0000 (16:09 +0200)
committerWolfgang Bumiller <w.bumiller@proxmox.com>
Thu, 23 Jun 2022 09:43:12 +0000 (11:43 +0200)
Azure AD seems to have a variable authorization code size, depending on
the browser state according to one report in bug #4074 [0].

Sometimes the size is greater than our current limit of 1024, so
increase it to 2048.

The RFC [1] mentions that there is no limit to the code size, but based on
current experience, a size limit of 2048 might be enough for every
current OpenID Connect provider.

[0] https://bugzilla.proxmox.com/show_bug.cgi?id=4074
[1] https://datatracker.ietf.org/doc/html/rfc6749#section-4.1.2

Signed-off-by: Mira Limbeck <m.limbeck@proxmox.com>
[w.bumiller@proxmox.com: bump to 4096]
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
src/PVE/API2/OpenId.pm

index c838b2cc2377accaff76ec45165025e1e544897f..77410e6d880402230603307122e793c219883d8c 100644 (file)
@@ -134,7 +134,7 @@ __PACKAGE__->register_method ({
            code => {
                description => "OpenId authorization code.",
                type => 'string',
-               maxLength => 1024,
+               maxLength => 4096,
             },
            'redirect-url' => {
                description => "Redirection Url. The client should set this to the used server url (location.origin).",