matty
/
peertube-theme-nctv-dark
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge branch 'master' into 'master' 

saml: parses role of user as int.

See merge request framasoft/peertube/official-plugins!14
This commit is contained in:
Chocobozzz 2022-04-20 06:51:36 +00:00
parent df650283b1 22d86fe574
commit d710d88ed6
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
peertube-plugin-auth-saml2/main.js
View File

@ -328,10 +328,16 @@ async function buildUser (settingsManager, samlUser) {
let username = findInUser(samlUser, settings['username-property']) || ''
username = username.replace(/[^a-z0-9._]/g, '_')
let sent_role = findInUser(samlUser, settings['role-property'])
let parsed_role = parseInt(sent_role, 10)
if (!Number.isSafeInteger(parsed_role)) {
parsed_role = undefined
}
return {
username,
email: findInUser(samlUser, settings['mail-property']),
displayName: findInUser(samlUser, settings['display-name-property']),
role: findInUser(samlUser, settings['role-property'])
role: parsed_role
}
}