Skip to content

User idle warning modal and auto logout (clear session) after configurable timeout value is hit - #4

Merged
chadpalmer merged 35 commits into
Developmentfrom
feedback-user-timeout
Apr 1, 2026
Merged

User idle warning modal and auto logout (clear session) after configurable timeout value is hit#4
chadpalmer merged 35 commits into
Developmentfrom
feedback-user-timeout

Conversation

@chadpalmer

@chadpalmer chadpalmer commented Mar 5, 2026

Copy link
Copy Markdown
Collaborator

This pull requests adds 1 new feature and 1 bug fix.

  1. Added customizable app admin settings that set the amount of idle time before the app auto logs users out of chat.
  2. The auto logout clears the session in chat, but does not sign the users out of their SSO accounts.
  3. One of the admin settings sets how long before the auto logout the warning dialog appears letting the users know that they will logged out due to inactivity.
  4. You can set the auto logout for 2 minutes of inactivity and the warning dialog to 1 minute for easy testing in the local Docker setup, but the admin settings variables can (and should be) adjusted for production. (30 minute idle time and 28 minute warning dialog for example)
  5. The current setup allows the simple movement of the mouse pointer over the OIG Chat window to reset the idle timer and hide the warning dialog without the need to explicitly click the "Stay signed in" button.
  6. BUG FIX: Fixed bug where settings object was muted in place and then compared to itself.

(NOTE: The code went through a lot of iterations implementing copilot's suggestions. The code iterations are documented in the docs/explanations/fixes section so you can see the reasoning behind the iterations.)

New warning modal window:
idle-warning-modal

Updated admin section:
admin-idle-timeout-settings

…ow horizontal scroll in response window while preserving access to drop down menus.
…s out of oig chat and clears app session after certain time of inactivity.
Merge Deveopment into feedback-user-timeout
@chadpalmer
chadpalmer requested review from Xeelee33 and Copilot March 5, 2026 17:39
@github-actions

github-actions Bot commented Mar 5, 2026

Copy link
Copy Markdown

📋 Release Notes Reminder

This PR appears to contain changes that should be documented in the release notes.

Reason: Feature-related keywords detected in PR title/body

📝 Please consider updating:

docs/explanation/release_notes.md

Template for new features:

*   **Feature Name**
    *   Brief description of the feature.
    *   **Key Details**: Important implementation notes.
    *   **Files Modified**: `file1.py`, `file2.js`.
    *   (Ref: related components, patterns)

Template for bug fixes:

*   **Bug Fix Title**
    *   Description of what was fixed.
    *   **Root Cause**: What caused the issue.
    *   **Solution**: How it was resolved.
    *   **Files Modified**: `file.py`.
    *   (Ref: related issue numbers, components)

This is an automated reminder. If this PR doesn't require release notes (e.g., internal refactoring, documentation-only changes), you can ignore this message.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@chadpalmer

chadpalmer commented Mar 5, 2026

Copy link
Copy Markdown
Collaborator Author

Release notes file is now updated with latest change info.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated 8 comments.

Comment thread application/single_app/route_frontend_authentication.py
Comment thread functional_tests/test_idle_logout_timeout.py Outdated
Comment thread application/single_app/app.py Outdated
Comment thread application/single_app/templates/base.html Outdated
Comment thread application/single_app/static/css/chats.css Outdated
Comment thread application/single_app/templates/base.html Outdated
Comment thread application/single_app/route_frontend_authentication.py Outdated
Comment thread application/single_app/route_frontend_authentication.py Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 4 comments.

Comment thread application/single_app/functions_settings.py
Comment thread application/single_app/route_frontend_authentication.py Outdated
Comment thread application/single_app/config.py Outdated
Comment thread docs/explanation/release_notes.md Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 14 out of 14 changed files in this pull request and generated 2 comments.

Comment thread application/single_app/templates/base.html Outdated
Comment thread application/single_app/route_frontend_admin_settings.py Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 16 out of 16 changed files in this pull request and generated 5 comments.

Comment thread functional_tests/test_settings_deep_merge_persistence_fix.py Outdated
Comment thread functional_tests/test_idle_logout_timeout.py Outdated
Comment thread functional_tests/test_admin_settings_safe_int_fallback_fix.py Outdated
Comment thread application/single_app/app.py Outdated
Comment thread docs/explanation/release_notes.md Outdated
@chadpalmer
chadpalmer requested a review from Copilot March 12, 2026 15:26

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 19 out of 19 changed files in this pull request and generated 2 comments.

Comment thread application/single_app/static/js/idle-logout-warning.js
Comment thread docs/explanation/fixes/IDLE_HEARTBEAT_INTERVAL_FIX.md Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 20 out of 20 changed files in this pull request and generated 2 comments.

Comment thread docs/explanation/release_notes.md Outdated
Comment thread docs/explanation/release_notes.md Outdated
@chadpalmer chadpalmer changed the title Feedback user timeout Added user idle warning modal and auto logout (clear session) after configurable timeout value is hit Mar 12, 2026
@chadpalmer chadpalmer changed the title Added user idle warning modal and auto logout (clear session) after configurable timeout value is hit User idle warning modal and auto logout (clear session) after configurable timeout value is hit Mar 13, 2026
@chadpalmer
chadpalmer force-pushed the feedback-user-timeout branch from a6d5c7d to 79d3e18 Compare March 31, 2026 21:55
@chadpalmer
chadpalmer merged commit 0527e8d into Development Apr 1, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants