Skip to content

Conversation

@smg6511
Copy link
Collaborator

@smg6511 smg6511 commented Dec 30, 2025

What does it do?

Maps the login window action "Security/Login" to the controller one "security/login" (lowercase).

Additionally, some minor css changes were made to the status message toast, as its content was being clipped with short messages (such as that which is shown after successfully extending a session).

Why is it needed?

Login is probably the only action where it can occur both directly via the controller (main login page) and via a window (extend session). The window, however, must use the newer (3.x) casing scheme in its action to call its corresponding processor. The core connector, however, was only testing for the presence of the lowercase formatted action to make certain decisions.

How to test

  1. Run grunt build to see the style changes
  2. Use one of two ways to force-expire your session: 1) change your session_cookie_lifetime system setting to a very low number like 60 (1 minute) and login with the remember me checkbox ticked; or 2) manually delete the session cookie from your browser's dev tools pane.
  3. After normal login, go to the Reports/Error Log page. Once you know your current session has expired click the refresh button; this should bring up the login window to extend your session. Verify that clicking login from there allows you to continue your work.

Related issue(s)/PR(s)

Resolves #16742

Special Note

The process of fixing the underlying cause of this issue raises another, which will be separately addressed. Some requests, like when trying to save a Resource after your session has expired, result in a mangle 401 response which prevents the login window from ever showing.

Fix conditional to allow login via processor-based request
Minor code formatting/quality fixes
Modernize and adjust message window defs to better accommodate short messages; short ones like the one for an extended session were clipped at the bottom previously
@smg6511 smg6511 added pr/review-needed Pull request requires review and testing. requires build Grunt build is required for integration labels Dec 30, 2025
@opengeek opengeek changed the title Fix login window (for extending an expired session) functionality Fix expired session login dialog Dec 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr/review-needed Pull request requires review and testing. requires build Grunt build is required for integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cannot login using the "Login expired" popup form.

1 participant