Skip to content

Commit 448c6ae

Browse files
committed
FIX puid not being stored in cwl data
The OID used for the puid was wrong. So I've set it to the right one. I've also updated the dev IDP so that it'll send out the puid attribute on that OID. I've also changed the eduPersonAffiliation to values that I see in staging. I noticed that the dev IDP doesn't have the SP metadata checked in, so I've added it in. Also noticed that nodeservices is probably very outdated, although it still seems to work. Added comments that it might not be necessary in the next LTS due to deprecations. Couldn't get it to work in dev, but seems to be working in prod, so not touching it right now.
1 parent 925f7f1 commit 448c6ae

File tree

4 files changed

+42
-10
lines changed

4 files changed

+42
-10
lines changed

LocalSettings.php

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -643,9 +643,8 @@ function loadenv($envName, $default = "") {
643643
# UBCAuth required attributes:
644644
# eduPersonAffiliation, an array of (staff, student, faculty, etc)
645645
'eduPersonAffiliationAttribute' => 'urn:oid:1.3.6.1.4.1.5923.1.1.1.1',
646-
# non-standard attributes, uncertain OIDs
647-
# ubc's puid
648-
'puidAttribute' => 'ubcEduCwlPuid',
646+
# ubc's puid, non-standard attribute/OID
647+
'puidAttribute' => 'urn:mace:dir:attribute-def:ubcEduCwlPuid',
649648
]
650649
];
651650

docker-compose.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -159,6 +159,9 @@ services:
159159
# - MEDIAWIKI_API_URL=http://web/w/api.php
160160
# - PARSOID_DOMAIN=localhost
161161
# - PARSOID_URL=http://parsoid:8000
162+
# parsoid is now integrated into mediawiki since 1.35
163+
# restbase is being deprecated
164+
# TODO: we might not need nodeservices anymore in the next lts
162165
nodeservices:
163166
image: ubcctlt/mediawiki-node-services
164167
ports:

docker/simplesamlphp/idp/config/authsources.php

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -100,30 +100,30 @@
100100
'student01:student01' => [
101101
'uid' => ['student01'],
102102
'displayName' => 'Student 01',
103-
'ubcEduCwlPuid' => 'PUIDST01',
104-
'eduPersonAffiliation' => ['member', 'student'],
103+
'urn:mace:dir:attribute-def:ubcEduCwlPuid' => 'PUIDST01',
104+
'eduPersonAffiliation' => ['student'],
105105
'mail' => '[email protected]'
106106
],
107107
'instructor01:instructor01' => [
108108
'uid' => ['instructor01'],
109109
'displayName' => 'Instructor 01',
110-
'ubcEduCwlPuid' => 'PUIDIN01',
110+
'urn:mace:dir:attribute-def:ubcEduCwlPuid' => 'PUIDIN01',
111111
'alt' => '51092d7f-2f38-4a91-bfb0-13a021c02df3',
112-
'eduPersonAffiliation' => ['member', 'student'],
112+
'eduPersonAffiliation' => ['faculty', 'student'],
113113
'mail' => '[email protected]'
114114
],
115115
'employee:employeepass' => [
116116
'uid' => ['employee'],
117117
'displayName' => 'Employee 00',
118-
'ubcEduCwlPuid' => 'PUIDEM00',
119-
'eduPersonAffiliation' => ['member', 'employee'],
118+
'urn:mace:dir:attribute-def:ubcEduCwlPuid' => 'PUIDEM00',
119+
'eduPersonAffiliation' => ['staff', 'alumni'],
120120
'mail' => '[email protected]'
121121
],
122122
# intended to simulate a basic CWL account
123123
'blockme01:blockme01' => [
124124
'uid' => ['blockme01'],
125125
'displayName' => 'Block Me01',
126-
'ubcEduCwlPuid' => 'PUIDBM01',
126+
'urn:mace:dir:attribute-def:ubcEduCwlPuid' => 'PUIDBM01',
127127
'eduPersonAffiliation' => [],
128128
'mail' => '[email protected]'
129129
],
Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
<?php
2+
3+
$metadata['http://wiki.docker:8080/_saml2'] = [
4+
'SingleLogoutService' => [
5+
[
6+
'Binding' => 'urn:oasis:names:tc:SAML:2.0:bindings:HTTP-Redirect',
7+
'Location' => 'http://wiki.docker:8080/_saml2/module.php/saml/sp/saml2-logout.php/wiki-sp',
8+
],
9+
],
10+
'AssertionConsumerService' => [
11+
[
12+
'Binding' => 'urn:oasis:names:tc:SAML:2.0:bindings:HTTP-POST',
13+
'Location' => 'http://wiki.docker:8080/_saml2/module.php/saml/sp/saml2-acs.php/wiki-sp',
14+
'index' => 0,
15+
],
16+
[
17+
'Binding' => 'urn:oasis:names:tc:SAML:2.0:bindings:HTTP-Artifact',
18+
'Location' => 'http://wiki.docker:8080/_saml2/module.php/saml/sp/saml2-acs.php/wiki-sp',
19+
'index' => 1,
20+
],
21+
],
22+
'contacts' => [
23+
[
24+
'emailAddress' => '[email protected]',
25+
'givenName' => 'UBC LT Hub',
26+
'contactType' => 'technical',
27+
],
28+
],
29+
'certData' => 'MIIEcTCCAtmgAwIBAgIUWmBx+tf9d7hKrFe9sjuhClKXFZ8wDQYJKoZIhvcNAQELBQAwSDELMAkGA1UEBhMCQ0ExEjAQBgNVBAcMCVZhbmNvdXZlcjEMMAoGA1UECgwDVUJDMRcwFQYDVQQDDA5zcC53aWtpLmRvY2tlcjAeFw0yNDA3MDQwOTA4MzZaFw0zNDA3MDQwOTA4MzZaMEgxCzAJBgNVBAYTAkNBMRIwEAYDVQQHDAlWYW5jb3V2ZXIxDDAKBgNVBAoMA1VCQzEXMBUGA1UEAwwOc3Aud2lraS5kb2NrZXIwggGiMA0GCSqGSIb3DQEBAQUAA4IBjwAwggGKAoIBgQDCEa0f5ZJhpSU+Xc0WNohbxTzpmDkqgI0rtWCmL5vqJakPCHnWnq0icCX2/zwh6//WP+9UPgO1ifHUhNC/NEJhBKGJjtNNKaV+AwUzj43IiLMqgkhMEvkqNePuKNBh/lvzjLl3KYMrLAEZKx+AluMaS7us5CmR9lyhY9nHZS0P1FRjwJ6SJ1o0HEuXHkH5eRotaRtrd8L+L93R9SaIBpgAy0XMkgFDqGmX7NbVAMT6cPNEVmj63J5veMtpCN5mQRXpZFPCSbmXOGlyy7S3cilpSk8QA8QOkt4EB+I6G5W/aaG8hNs4QHKkKMReJ/oHQbQXIJ2d4oMsQaEXk3FtTIbl4l7fKS+LvhCHvB9z8q/ueh3bAIcpSxGzg3oTScZM5ZZAqzjYxCMYdI+3h44FPUtDsZdwezFN/B+JsITQouaYzuRxjUV6uNGhZXSRb+st3VYIBg0+mIvowDyBHgQvOaAZ8/UuSqcfrMH/AwTVY2Ej2YzerKDCwchHmpv5sXRY+o8CAwEAAaNTMFEwHQYDVR0OBBYEFIUt4n/0ouPzNfRNonY/EtJhHXPfMB8GA1UdIwQYMBaAFIUt4n/0ouPzNfRNonY/EtJhHXPfMA8GA1UdEwEB/wQFMAMBAf8wDQYJKoZIhvcNAQELBQADggGBAAK5QNOmFjLmQZdfWURK+hyCN08RIB6qOgKxuMG6j6u4brKOhktRAx+8hwrgVH96+fW3DkELsNGTTjUzxJvXM01cDDn2lUNMhLA2InHTsFe2zbmKG5sSl0wOFhi0kBnkGL8di3FgnqJJs8sTcQWajoFiEPa0yW3Gad/S6JSPgrHMlPkMPgZ8Vw8aYVprronbj9eiGWzRO5vFrE6YMn2l9es/pVJKzsb362EPhFekJA6f+6Ek2rfPRd0KiF5+Pln8KSooRmXpOZkM2CUfgOmb3lT9mwel2wemnXjUj0sjN5luotbK6YVhnwuq9d1O1a8Lhx8HLLasV7bR1hg9rjz+K2nv1XqWYsiFJelkgD4DOcFP68I/eiUiAf6jqh5+YJuqFXkXS9P6ohOXn5sbiV69+VV64JXG31emPgX/mm/41Bq2j5ESYak1I4RCPdLPpsjPWUMUKAXrRjbj8UZBf5w3Uv7tc4SY+Sc8mcBw0/14Ossz5h2ZLBW0j1QKqDWwSyWn5A==',
30+
];

0 commit comments

Comments
 (0)