Bug 32897 - UCS-overview changes impact saml app
UCS-overview changes impact saml app
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: SAML
UCS 3.2
Other Linux
: P5 normal (vote)
: UCS 3.2
Assigned To: Erik Damrose
Felix Botner
: interim-3
Depends on: 32613 32932
Blocks:
  Show dependency treegraph
 
Reported: 2013-10-18 09:49 CEST by Erik Damrose
Modified: 2013-11-19 06:42 CET (History)
1 user (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Erik Damrose univentionstaff 2013-10-18 09:49:35 CEST
Recent changes to the ucs-overview page and its javascript rendered the language switcher useless on the single-sign-on page.

In addition, the links in the footer changed and have to be updated.
Comment 1 Erik Damrose univentionstaff 2013-10-18 09:50:55 CEST
It should be discussed if the updated links in the footer will be released in a new app version for UCS 3.1-1
Comment 2 Stefan Gohmann univentionstaff 2013-10-18 18:47:35 CEST
(In reply to Erik Damrose from comment #1)
> It should be discussed if the updated links in the footer will be released
> in a new app version for UCS 3.1-1

I don't think we need an update for UCS 3.1.
Comment 3 Erik Damrose univentionstaff 2013-10-22 12:33:54 CEST
Changes to the ucs-overview have been incorporated.

Fixed in
univention-saml 1.0.6-1.19.201310221151

A new App version (v1.2) is available in the test appcenter.
Comment 4 Felix Botner univentionstaff 2013-10-24 10:39:27 CEST
goto saml login page, perform wrong login and change the language -> boom
Comment 5 Felix Botner univentionstaff 2013-10-24 10:43:55 CEST
languages are mixed up after login with a "non-saml" user 

Zugriff verboten
Sie haben nicht die erforderlichen Berechtigungen um auf diese Anwendung zuzugreifen. Bitte kontaktieren Sie einen Administrator, wenn Sie dies ungerechtfertigt finden.
Logout


and

Access forbidden
You don't have the needed privileges to access this application. Please contact the administrator if you find this to be incorrect.
Abmelden
Comment 6 Erik Damrose univentionstaff 2013-10-24 17:07:06 CEST
Languages should not be mixed up anymore. 

A language switch after entering wrong credentials is not possible, even the default simplesamltheme does not offer the option. This is due to the fact that the current language switch implementation causes the HTTP POST parameters to be discarded, which hold the session information.

New packages are available in the test-appcenter, app version 1.2,
univention-saml 1.0.7-1.20.201310241626
Comment 7 Felix Botner univentionstaff 2013-10-25 10:46:23 CEST
OK - overview
OK - no changelog entry needed
Comment 8 Erik Damrose univentionstaff 2013-10-31 17:26:32 CET
The updates at Bug #32613 broke the design again.
Comment 9 Erik Damrose univentionstaff 2013-11-01 11:38:07 CET
Updated footer links, new package version available in the test appcenter 

univention-saml 1.0.8-1.21.201311011104
Comment 10 Felix Botner univentionstaff 2013-11-01 13:03:00 CET
I can see the "Administration" and "Webservices" tabs on the saml login page.
Comment 11 Erik Damrose univentionstaff 2013-11-01 13:39:35 CET
The problem only existed in firefox. An additional css rule in the header template fixed it.

univention-saml 1.0.9-1; test appcenter is updated
Comment 12 Felix Botner univentionstaff 2013-11-01 15:29:06 CET
saml login site, select english, login with invalid password -> german error message (only in firefox)
Comment 13 Erik Damrose univentionstaff 2013-11-04 14:56:53 CET
The change to let JS handle language switching and thereby reduce the work needed when layout/translation changes occur did not work out. Translations and the respective layout changes are now completely handled by univention-saml.

The test appcenter has been updated:
r45832 univention-saml 1.0.10-1.23.201311041442
Comment 14 Felix Botner univentionstaff 2013-11-04 16:47:17 CET
OK
Comment 15 Stefan Gohmann univentionstaff 2013-11-19 06:42:51 CET
UCS 3.2 has been released:
 http://docs.univention.de/release-notes-3.2-en.html
 http://docs.univention.de/release-notes-3.2-de.html

If this error occurs again, please use "Clone This Bug".