Skip to content

feat(auth): replace expires_in with expires_at for token expiry#199

Merged
deo002 merged 1 commit intofossology:mainfrom
aaryan359:feat/auth-add-access-token-expires-at
Feb 13, 2026
Merged

feat(auth): replace expires_in with expires_at for token expiry#199
deo002 merged 1 commit intofossology:mainfrom
aaryan359:feat/auth-add-access-token-expires-at

Conversation

@aaryan359
Copy link
Copy Markdown
Contributor

Description

As discussed in this fossology/LicenseDb-UI#26, I updated the expire_in logic to expire_at, and also created a PR in the frontend fossology/LicenseDb-UI#54

Changes

  • Added backend-generated expires_at field for access tokens
  • Removed reliance on client-side expiry calculation using expires_in
  • Ensured token expiry is authoritative and latency-safe

Submitter Checklist

  • Includes tests
  • [x ] Includes docs
  • I have tested my changes locally.

References

N/A

@aaryan359
Copy link
Copy Markdown
Contributor Author

@deo002 Please check this? Does it looks good?

Copy link
Copy Markdown
Collaborator

@deo002 deo002 left a comment

Choose a reason for hiding this comment

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

Thanks for your contributions! Some changes are required.

Signed-off-by: aaryan359 <aaryanmeena96@gmail.com>
@aaryan359 aaryan359 force-pushed the feat/auth-add-access-token-expires-at branch from e9aa01e to c36df1d Compare February 11, 2026 15:52
@aaryan359 aaryan359 requested a review from deo002 February 11, 2026 16:00
@aaryan359
Copy link
Copy Markdown
Contributor Author

@deo002 , Please have a look!

Copy link
Copy Markdown
Collaborator

@deo002 deo002 left a comment

Choose a reason for hiding this comment

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

Changes look good! Working as expected. Thanks for your contributions.

@deo002 deo002 merged commit 2468790 into fossology:main Feb 13, 2026
10 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