Skip to content

Commit 6a05fb5

Browse files
Add upper-bound to social-auth-core (#711)
Authentication breaks when we go to 4.5.5, so this fixes that AAP-43108 # [AAP-43108] Brief Title Describing the Change ## Description <!-- Mandatory: Provide a clear, concise description of the changes and their purpose --> - What is being changed? Pin social core to 4.5.4 and below. - Why is this change needed? Authentication breaks with our FakeBackend class on any version above social_core 4.5.5 - How does this change address the issue? Pins social core to 4.5.4 and below, so we do not end up using 4.5.5 ## Type of Change - [X] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] Documentation update - [ ] Test update - [ ] Refactoring (no functional changes) - [ ] Development environment change - [ ] Configuration change ## Self-Review Checklist - [X] I have performed a self-review of my code - [X] I have added relevant comments to complex code sections - [ ] I have updated documentation where needed - [X] I have considered the security impact of these changes - [X] I have considered performance implications - [X] I have thought about error handling and edge cases - [X] I have tested the changes in my local environment ## Testing Instructions Start up an test_app, ensure dependencies are reinstalled and do not cause a conflict ### Expected Results No dependency conflicts ## Additional Context <!-- Optional but helpful information --> ### Required Actions <!-- Check if changes require work in other areas -> <!-- Remove section if no external actions needed --> - [ ] Requires documentation updates <!-- API docs, feature docs, deployment guides --> - [ ] Requires downstream repository changes <!-- Specify repos: django-ansible-base, eda-server, etc. --> - [ ] Requires infrastructure/deployment changes <!-- CI/CD, installer updates, new services --> - [ ] Requires coordination with other teams <!-- UI team, platform services, infrastructure --> - [ ] Blocked by PR/MR: #XXX <!-- Reference blocking PRs/MRs with brief context --> ### Screenshots/Logs <!-- Add if relevant to demonstrate the changes -->
1 parent 81fe6f2 commit 6a05fb5

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

requirements/requirements_authentication.in

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
social-auth-app-django==5.4.1
2+
social-auth-core<=4.5.4
23
tabulate
34

45
# These should eventually be split out when the authentications move into their own repo

0 commit comments

Comments
 (0)