Skip to content

temp: Constrain social-auth packages for POST issue#552

Merged
feanil merged 1 commit into
masterfrom
timmc/post-drf
Jul 9, 2026
Merged

temp: Constrain social-auth packages for POST issue#552
feanil merged 1 commit into
masterfrom
timmc/post-drf

Conversation

@timmc-edx

Copy link
Copy Markdown
Contributor

See openedx/edx-drf-extensions#561

Merge checklist:

  • If adding new checks, followed how_tos/0001-adding-new-checks.rst
  • Changelog record added (if needed)
  • Documentation updated (not only docstrings) (if needed)
  • Commits are squashed

@timmc-edx timmc-edx requested review from feanil and kdmccormick and removed request for kdmccormick July 7, 2026 18:25
@feanil

feanil commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

@timmc-edx this change makes sense to mitigate the issue introduced by the upgrade. But it's not clear to me what the long term solution is to fix this? I see that in openedx/edx-drf-extensions#561 there is a proposed solution, is anyone actually working on that?

@timmc-edx

Copy link
Copy Markdown
Contributor Author

No one is currently working on it -- our team is finishing up some edxapp containerization work and will likely not be looking into edx-drf-extensions fixes within a month time frame.

But I also haven't looked into the details enough to assess whether the proposed fixes are what we want, or whether we should instead be pushing back on the social-auth changes, etc.

@feanil feanil merged commit 8636805 into master Jul 9, 2026
5 checks passed
@feanil feanil deleted the timmc/post-drf branch July 9, 2026 15:41
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