View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0033426 | mantisbt | authentication | public | 2023-12-12 05:37 | 2024-02-20 16:58 |
Reporter | silambu | Assigned To | dregad | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 2.26.0 | ||||
Target Version | 2.26.1 | Fixed in Version | 2.26.1 | ||
Summary | 0033426: User not authenticated when following link from notification email | ||||
Description | Hello , please find the screenshot for reference , I have clicked my mantis ticket link from email, generally it should open in new tab automatically , Here i need to click login or simply hitting the enter the session page was getting loaded and opening . Thank you | ||||
Tags | No tags attached. | ||||
Attached Files | |||||
Sounds like normal behavior caused by expired user session / login cookie. Just complete the login process. |
|
@dregad there is a changed behavior in 2.26.0. IIRC this was different in older versions. |
|
We also experiencing this problem. You can enter whatever you like in the login field or even just click login button with empty field. |
|
Thanks for the pointer @atrol I performed some quite tedious testing and analysis using git bisect, and indeed I can confirm the problem now. It has been present for quite some time, but I never really noticed or paid attention to it. The offending commit is MantisBT master ea80bcfb see 0029611 Problem is the new setting for the Cookie SameSite attribute being set to Strict by default (used to be None), so the cookie is not sent for requests originating from a different domain. |
|
@silambu @tanidar @atrol I see now that this is detrimental to UX in a quite common usage scenario (click link from email), so maybe the default should be set to Lax instead. And of course improve the documentation in Admin Guide, to clearly describe the impact of each value on system behavior. What do you think ? |
|
@dregad thanks, that worked for us. We using Mantis with Trello as task management so that was very irritant issue. |
|
@dregad seems to make sense to me |
|
Thanks for your feedback. |
|
Thanks for the workaround, It's works for me :) |
|
MantisBT: master 14e7eccb 2024-01-06 08:02 Details Diff |
Change $g_cookie_samesite default to 'Lax' The original value was 'Strict' for security purposes, the intention being to provide the strongest possible protection against CSRF attacks. Unfortunately, this actually prevents the user's session cookie from being recognized when clicking a link from a notification email, causing MantisBT to open an anonymous session even when the user is logged in. Changing the default value to 'Lax' fixes the issue. Fixes 0033426 |
Affected Issues 0033426 |
|
mod - config_defaults_inc.php | Diff File | ||
MantisBT: master-2.26 d6febe31 2024-01-06 08:02 Details Diff |
Change $g_cookie_samesite default to 'Lax' The original value was 'Strict' for security purposes, the intention being to provide the strongest possible protection against CSRF attacks. Unfortunately, this actually prevents the user's session cookie from being recognized when clicking a link from a notification email, causing MantisBT to open an anonymous session even when the user is logged in. Changing the default value to 'Lax' fixes the issue. Fixes 0033426 (cherry picked from commit 14e7eccb07abb8def19968da9118b7dc89996efa) |
Affected Issues 0033426 |
|
mod - config_defaults_inc.php | Diff File | ||
MantisBT: master 9d9d4c21 2024-01-06 08:21 Details Diff |
Document effect of $g_cookie_samesite = 'Strict' Also explain that users with an active session cookie need to log out and back in if this setting is changed. Fixes 0033426 |
Affected Issues 0033426 |
|
mod - config_defaults_inc.php | Diff File | ||
mod - docbook/Admin_Guide/en-US/config/cookies.xml | Diff File | ||
MantisBT: master-2.26 59e96cea 2024-01-06 08:21 Details Diff |
Document effect of $g_cookie_samesite = 'Strict' Also explain that users with an active session cookie need to log out and back in if this setting is changed. Fixes 0033426 (cherry picked from commit 9d9d4c21395bba60002c1c0059862c936757b300) |
Affected Issues 0033426 |
|
mod - config_defaults_inc.php | Diff File | ||
mod - docbook/Admin_Guide/en-US/config/cookies.xml | Diff File |