Skip to content

docs: Give explicit type for allow_multiple_user_backends app config#1219

Merged
julien-nc merged 1 commit intomainfrom
docs/type-allow_multiple_user_backends-config
Oct 13, 2025
Merged

docs: Give explicit type for allow_multiple_user_backends app config#1219
julien-nc merged 1 commit intomainfrom
docs/type-allow_multiple_user_backends-config

Conversation

@provokateurin
Copy link
Copy Markdown
Member

The app was broken for me with

{"reqId":"DihuHRmsPwO0aumqJxz2","level":2,"time":"2025-10-10T17:53:13+00:00","remoteAddr":"2a02:8071:67f1:12e0:54ea:e5e5:e86c:c644","user":"--","app":"user_oidc","method":"GET","url":"/login","message":"conflict with value type from database","userAgent":"Mozilla/5.0 (X11; Linux x86_64; rv:143.0) Gecko/20100101 Firefox/143.0","version":"31.0.9.1","data":{"app":"user_oidc","key":"allow_multiple_user_backends","type":"4","knownType":"8"}}

which I believe is a regression from d440778

My previous command was occ config:app:set user_oidc allow_multiple_user_backends --type integer --value 0 because I assumed the type would be an integer.
The readme should therefore mention explicitly that it's a string, as it's currently only implicit, but still required.

Depending on the number of instances that break with this change, you might want to consider changing the type to be an integer, as it currently only seems to be used with 0 or 1 as the value.

Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin provokateurin added bug Something isn't working documentation Improvements or additions to documentation 3. to review labels Oct 10, 2025
@provokateurin
Copy link
Copy Markdown
Member Author

CI failure is unrelated.

@julien-nc julien-nc merged commit 375cbdb into main Oct 13, 2025
63 of 71 checks passed
@julien-nc julien-nc deleted the docs/type-allow_multiple_user_backends-config branch October 13, 2025 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review bug Something isn't working documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants