Skip to content

config: update REFRESH_TOKEN_EXPIRE_SECONDS to 30 days#4419

Merged
Anas12091101 merged 2 commits intomainfrom
anas/add-REFRESH_TOKEN_EXPIRE_SECONDS-edx
Apr 21, 2026
Merged

config: update REFRESH_TOKEN_EXPIRE_SECONDS to 30 days#4419
Anas12091101 merged 2 commits intomainfrom
anas/add-REFRESH_TOKEN_EXPIRE_SECONDS-edx

Conversation

@Anas12091101
Copy link
Copy Markdown
Contributor

What are the relevant tickets?

https://github.com/mitodl/hq/issues/10116

Description (What does it do?)

This PR updates the REFRESH_TOKEN_EXPIRE_SECONDS in edX to 30 days, aligning it with other apps.

Screenshots (if appropriate):

  • Desktop screenshots
  • Mobile width screenshots

How can this be tested?

Additional Context

Copilot AI review requested due to automatic review settings April 1, 2026 08:06
Copy link
Copy Markdown
Contributor

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

Updates the Open edX (edxapp) OAuth refresh-token lifetime configuration to match the intended 30-day expiration window.

Changes:

  • Changed OAUTH2_PROVIDER.REFRESH_TOKEN_EXPIRE_SECONDS from 90 days (7,776,000s) to 30 days (2,592,000s).
  • Added an inline comment clarifying the duration (“30 days”).

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

Comment thread src/ol_infrastructure/applications/edxapp/config_builder.py
@Anas12091101 Anas12091101 marked this pull request as draft April 1, 2026 08:18
@Anas12091101
Copy link
Copy Markdown
Contributor Author

Converted the PR to a draft since we’ll need to bump the ol-social-auth version after adding the task in the plugin. I think it makes sense to include that in this PR as well.

@Anas12091101
Copy link
Copy Markdown
Contributor Author

We should wait for mitodl/open-edx-plugins#778 to be merged before merging this PR.

@Anas12091101 Anas12091101 marked this pull request as ready for review April 21, 2026 12:12
@Anas12091101 Anas12091101 merged commit 87a56b8 into main Apr 21, 2026
8 checks passed
@Anas12091101 Anas12091101 deleted the anas/add-REFRESH_TOKEN_EXPIRE_SECONDS-edx branch April 21, 2026 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants