sorbaugh
commented
2 months ago Edited description. Unfortunately this isn't only limited to 27.
Open GretaD opened 2 months ago
sorbaugh
commented
2 months ago Edited description. Unfortunately this isn't only limited to 27.
kesselb
commented
2 months ago
⚠️ This issue respects the following points: ⚠️
Bug description
In NC27 with activated theming app and custom logo and backgroud I am getting the default theming if a saml user is deactivated and tries to login.
Steps to reproduce
Analysis:
In base.php handleLogin() is called, and inside there OC_User::handleApacheAuth() is called. Which again calls its own method loginWithApache(). There we have a check whether the user is enabled: if ($userSession->getUser() && !$userSession->getUser()->isEnabled()) {
In our case they are not enable and so a LoginException is thrown with the translated message "User disabled".
This then is only caught in index.php where it prints the error page. OC_Template::printErrorPage() is utilized. It actually would load theming, if it is not already, provided it is enabled for the user! But without a user – login refused - this check returns false and so we have not theming in that case.
Expected behavior
Theming applies
Installation method
None
Nextcloud Server version
27
Operating system
None
PHP engine version
None
Web server
None
Database engine version
None
Is this bug present after an update or on a fresh install?
None
Are you using the Nextcloud Server Encryption module?
None
What user-backends are you using?
Configuration report
No response
List of activated Apps
No response
Nextcloud Signing status
No response
Nextcloud Logs
No response
Additional info
No response