Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update dependency social-auth-app-django to v5 [SECURITY] #2328

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Aug 6, 2024

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
social-auth-app-django ^4.0.0 -> ^5.0.0 age adoption passing confidence

GitHub Vulnerability Alerts

CVE-2024-32879

Impact

Due to default case-insensitive collation in MySQL or MariaDB databases, third-party authentication user IDs are not case-sensitive and could cause different IDs to match.

Patches

This issue has been addressed by https://github.com/python-social-auth/social-app-django/pull/566 and fix released in 5.4.1.

Workarounds

An immediate workaround would be to change collation of the affected field:

ALTER TABLE `social_auth_usersocialauth` MODIFY `uid` varchar(255) COLLATE `utf8_bin`;

References

This issue was discovered by folks at https://opencraft.com/.


Release Notes

python-social-auth/social-app-django (social-auth-app-django)

v5.4.1

Compare Source

Changed
  • Added reverse migration for JSON field
  • Fixed improper handling of case sensitivity with MySQL/MariaDB (CVE-2024-32879)

v5.4.0

Compare Source

Changed
  • Improved JSON field migration performance
  • Introduce configuration to request POST only requests for social authentication
  • Updated list of supported Django and Python versions

v5.3.0

Compare Source

Changed
  • Uses Django native JSON field

v5.2.0

Compare Source

Changed
  • Removed support for Django<3.2
  • Fixed missing migration issue

v5.1.0

Compare Source

Changed
  • Compatibility with recent Django and Python versions
  • Coding style improvements
  • Improved error handling in SocialAuthExceptionMiddleware

v5.0.0

Compare Source

Changed
  • Removed compat shims for obsolete Django versions
  • Switch from deprecated django.conf.urls.url to django.urls.path
  • Use query .exists() instead of .count() > 0
  • Added testing for Django 3.0
  • Drop support for Python 2
  • Django generic JSONField support, details documented here
  • Django 3.2+ compatibility
  • Use _default_manager instead of objects

Configuration

📅 Schedule: Branch creation - "" in timezone US/Eastern, Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

Copy link
Contributor Author

renovate bot commented Aug 6, 2024

⚠️ Artifact update problem

Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.

♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

  • any of the package files in this branch needs updating, or
  • the branch becomes conflicted, or
  • you click the rebase/retry checkbox if found above, or
  • you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:

File name: poetry.lock
Updating dependencies
Resolving dependencies...


Because mitol-django-authentication (1.6.0) depends on djoser (2.1.0)
 and djoser (2.1.0) depends on social-auth-app-django (>=4.0.0,<5.0.0), mitol-django-authentication (1.6.0) requires social-auth-app-django (>=4.0.0,<5.0.0).
So, because mitx online depends on both mitol-django-authentication (1.6.0) and social-auth-app-django (^5.0.0), version solving failed.

@renovate renovate bot force-pushed the renovate/pypi-social-auth-app-django-vulnerability branch 17 times, most recently from 584bd14 to e418d6c Compare August 13, 2024 20:24
@renovate renovate bot force-pushed the renovate/pypi-social-auth-app-django-vulnerability branch 7 times, most recently from d9ecce7 to 079a12a Compare August 20, 2024 17:32
@renovate renovate bot force-pushed the renovate/pypi-social-auth-app-django-vulnerability branch 5 times, most recently from 8a679fa to abd42a1 Compare August 27, 2024 15:55
@renovate renovate bot force-pushed the renovate/pypi-social-auth-app-django-vulnerability branch 11 times, most recently from e058010 to b01a446 Compare September 3, 2024 16:38
@renovate renovate bot force-pushed the renovate/pypi-social-auth-app-django-vulnerability branch 4 times, most recently from 34557f0 to 6b4f306 Compare September 6, 2024 17:19
@renovate renovate bot force-pushed the renovate/pypi-social-auth-app-django-vulnerability branch 6 times, most recently from b3fb020 to 7beec82 Compare September 17, 2024 14:06
@renovate renovate bot force-pushed the renovate/pypi-social-auth-app-django-vulnerability branch 2 times, most recently from a494379 to 19d4008 Compare September 18, 2024 20:09
@renovate renovate bot force-pushed the renovate/pypi-social-auth-app-django-vulnerability branch from 19d4008 to 4fd624b Compare September 18, 2024 20:53
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.

0 participants