Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Polling system messages should not extend the session #3628

Closed
edmundoa opened this issue Mar 16, 2017 · 0 comments
Closed

Polling system messages should not extend the session #3628

edmundoa opened this issue Mar 16, 2017 · 0 comments
Assignees
@joschi
Labels
bug
Milestone
2.3.0

Comments

@edmundoa
Copy link
Contributor

Expected Behaviour

Leaving the system overview page open should comply with the user's session timeout settings.

Current Behaviour

Leaving the system overview page open do not cancel the users' session, as polling the system messages (/api/system/messages) is extending the session.

Steps to Reproduce (for bugs)

  1. Log in and go to System -> Overview
  2. Wait without any interaction
  3. The user is never logged out

Your Environment

  • Graylog Version: Graylog 2.3.0-SNAPSHOT (4802bbb), Graylog 2.2.3-SNAPSHOT is also affected
@edmundoa edmundoa added the bug label Mar 16, 2017
joschi pushed a commit that referenced this issue Mar 17, 2017
Prevent session extension for polling system messages
89a5f98 
Fixes #3628
@joschi joschi self-assigned this Mar 17, 2017
@joschi joschi added this to the 2.3.0 milestone Mar 17, 2017
@joschi joschi mentioned this issue Mar 17, 2017
Merged
@ghost ghost removed the in progress label Mar 17, 2017
edmundoa pushed a commit that referenced this issue Mar 17, 2017
@joschi @edmundoa
Prevent session extension for polling system messages (#3632)
41be168 
Fixes #3628
edmundoa pushed a commit that referenced this issue Mar 17, 2017
@joschi
Prevent session extension for polling system messages (#3632)
e2cbc71 
Fixes #3628
(cherry picked from commit 41be168)
@edmundoa edmundoa mentioned this issue Mar 17, 2017
Merged
joschi pushed a commit that referenced this issue Mar 17, 2017
@edmundoa @joschi
Prevent session extension for polling system messages (#3638)
83cd35e 
Refs #3632
Fixes #3628
(cherry picked from commit 41be168)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@joschi joschi

Labels
bug
Projects
None yet
Milestone
2.3.0
Development

No branches or pull requests
2 participants
@joschi @edmundoa