Skip to content

Use session duration to calculate session timeout#5446

Merged
greenbonebot merged 1 commit into
mainfrom
session-timeout
Jul 24, 2026
Merged

Use session duration to calculate session timeout#5446
greenbonebot merged 1 commit into
mainfrom
session-timeout

Conversation

@bjoernricks

@bjoernricks bjoernricks commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

What

Use session duration to calculate session timeout

THIS CHANGE REQUIRES A NEW GSAD VERSION!

Why

When the system time of the machine running gsad and the users system time where the browser is running GSA is not in sync the session timeout has been displayed incorrectly. Either it did show 0:00 and the session was still valid or the user has been logged out early. With this change the session timeout is calculated by getting the duration instead of the end time.

References

https://jira.greenbone.net/browse/GEA-1743
greenbone/gsad#433

Checklist

  • Tests

THIS REQUIRES A NEW GSAD VERSION!

When the system time of the machine running gsad and the users system
time where the browser is running GSA is not in sync the session timeout
has been displayed incorrectly. Either it did show 0:00 and the session
was still valid or the user has been logged out early. With this change
the session timeout is calculated by getting the duration instead of the
end time.
@bjoernricks
bjoernricks requested a review from a team as a code owner July 24, 2026 12:15
@greenbonebot
greenbonebot enabled auto-merge (rebase) July 24, 2026 12:15
@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA ea12d02.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@codecov

codecov Bot commented Jul 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 81.28%. Comparing base (36a0d1c) to head (ea12d02).

Files with missing lines Patch % Lines
src/gmp/commands/user.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5446      +/-   ##
==========================================
- Coverage   81.28%   81.28%   -0.01%     
==========================================
  Files        1280     1280              
  Lines       33322    33321       -1     
  Branches     9920     9919       -1     
==========================================
- Hits        27085    27084       -1     
  Misses       5565     5565              
  Partials      672      672              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@greenbonebot
greenbonebot merged commit ffcfb18 into main Jul 24, 2026
26 of 27 checks passed
@greenbonebot
greenbonebot deleted the session-timeout branch July 24, 2026 13:05
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.

3 participants