Rob Commeren (7e21ce30) at 11 Feb 11:05
Merge branch 'remove-redundant-security-checks' into 'master'
Rob Commeren (c651cdaa) at 11 Feb 07:59
Causes problems for SSO users. Fix version bumped.
/closes #1
Marijn van Wezel (7e21ce30) at 11 Feb 07:59
Merge branch 'remove-redundant-security-checks' into 'master'
... and 4 more commits
Causes problems for SSO users. Fix version bumped.
/closes #1
Done in commit c651cdaa
Rob Commeren (c651cdaa) at 04 Feb 08:57
Remove function override
Zie Slack :)
In gesprek is besloten om dit niet te doen. De security check is niet belangrijk genoeg en er zijn specifieke rechten voor nodig om op deze pagina's te komen, dus de check gaat er helemaal uit.
Zal ik doen. Is dat bevestiging dat je bovenstaande suggestie niets vind?
Since we are no longer using getLoginSecurityLevel
here, please also remove the implementation from any special pages that implement it (I think that is src/Special/SpecialActiveSpaces.php
, src/Special/SpecialAddSpace.php
and src/Special/SpecialArchivedSpaces.php
).
Rob Commeren (277b3ca5) at 04 Feb 08:15
Minor version update
Dit zou een minor release moeten zijn, omdat er functionaliteit is aangepast.
@marijn Het is ook mogelijk om dit achter een setting te plaatsen, die standaard wel aan staat. $wgWSSpacesSecurityCheck = true;
, maar kan op false gezet worden, dan is er geen extra check.
Laat maar even weten of je die aanpak beter vind, dan schrijf ik het om.
Causes problems for SSO users. Fix version bumped.
/closes #1
Rob Commeren (4f822e72) at 03 Feb 12:56
Fix version bump
Rob Commeren (a6f1750d) at 03 Feb 12:54
Remove security check from special pages
Rob Commeren (8dd29515) at 03 Feb 12:34