Univention Bugzilla – Full Text Bug Listing |
Summary: | Hinweis wenn Cookies nicht aktiviert sind | ||
---|---|---|---|
Product: | UCS | Reporter: | Alexander Kläser <klaeser> |
Component: | UMC (Generic) | Assignee: | Jürn Brodersen <brodersen> |
Status: | CLOSED FIXED | QA Contact: | Alexander Kläser <klaeser> |
Severity: | normal | ||
Priority: | P5 | CC: | best, brodersen |
Version: | UCS 4.1 | Flags: | klaeser:
Patch_Available+
|
Target Milestone: | UCS 4.1-2-errata | ||
Hardware: | Other | ||
OS: | Linux | ||
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): | Usability | |
Max CVSS v3 score: | |||
Attachments: | proposed patch |
Description
Alexander Kläser
2012-09-28 10:46:29 CEST
Created attachment 7663 [details]
proposed patch
(In reply to Jürn Brodersen from comment #1) > Created attachment 7663 [details] > proposed patch Patch looks nice :) ! Please commit the change. Little adjustment for the title: "Please enable cookies" "Bitte Cookies aktivieren" ... and for the message: "Browser cookies are necessary for working with Univention Management Console. Please activate cookies in your browser." "Browser-Cookies sind notwendig für die Arbeit mit Univention Management Console. Bitte aktivieren sie Cookies in Ihrem Browser." UCS4.1-2: r69849: * Show a warning on the login page if cookies are disabled r69850: YAML Package: univention-management-console-frontend Version: 5.0.63-44.1237.201606061440 IE 11 → FAILED EDGE 13 → FAILED Chromium 41 → OK Safari → OK Please check again for IE/Edge. UCS4.1-2: r70822: * IE did not show the cookies disabled warning r70823: YAML Package: univention-management-console-frontend Version: 5.0.63-46.1238.201607050936 (In reply to Jürn Brodersen from comment #5) > UCS4.1-2: > > r70822: * IE did not show the cookies disabled warning > r70823: YAML > Package: univention-management-console-frontend > Version: 5.0.63-46.1238.201607050936 My tests with IE 11 and Edge 13 failed again. The cause for this is has('ie'): | IE 11 | Edge 13 --------------------------------- has('ie') | false | false has('trident') | true | false has('edge') | false | true As discussed, you can additionally check whether umcUsername or umcSessionID are set as cookies. (In reply to Alexander Kläser from comment #7) > As discussed, you can additionally check whether umcUsername or umcSessionID > are set as cookies. I don't understand this. Checking for what? (In reply to Florian Best from comment #8) > (In reply to Alexander Kläser from comment #7) > > As discussed, you can additionally check whether umcUsername or umcSessionID > > are set as cookies. > I don't understand this. Checking for what? If umcUsername is already set as a cookie -> cookies must be enabled. If umUsername is not set, cookies might be disabled or the user was not logged in in the last ~90 days. umcSessionID has a longer time until it expires but gets removed on a logout. UCS4.1-2: r70830: * Check for existing cookie instead of IE Package: univention-management-console-frontend Version: 5.0.63-47.1239.201607051419 Changes: OK, works fine now also with IE11/Edge13 YAML file: OK, I adjusted the package version [r70857] → VERIFIED |