Skip to content

Increase access token lifetime to 1 hour#474

Merged
Flaminel merged 1 commit intomainfrom
fix_token_lifetime
Feb 27, 2026
Merged

Increase access token lifetime to 1 hour#474
Flaminel merged 1 commit intomainfrom
fix_token_lifetime

Conversation

@Flaminel
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Feb 27, 2026

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...rr.Api/Features/Auth/Controllers/AuthController.cs 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request increases the JWT access token lifetime from 1 minute to 1 hour to improve user experience by reducing the frequency of token refreshes. The change is made in two locations to maintain consistency between the token generation logic and the API response.

Changes:

  • Increased AccessTokenLifetime constant from 1 minute to 1 hour in JwtService
  • Updated ExpiresIn field in token response from 60 seconds to 3600 seconds to match the new lifetime

Reviewed changes

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

File Description
code/backend/Cleanuparr.Infrastructure/Features/Auth/JwtService.cs Updated AccessTokenLifetime constant from 1 minute to 1 hour
code/backend/Cleanuparr.Api/Features/Auth/Controllers/AuthController.cs Updated ExpiresIn field in TokenResponse from 60 to 3600 seconds

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Flaminel Flaminel merged commit 2333c86 into main Feb 27, 2026
11 of 12 checks passed
@Flaminel Flaminel deleted the fix_token_lifetime branch February 27, 2026 13:13
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