Update requirements.txt for social-auth-core to use 'all' extras #6028

Closed
opened 2025-12-29 19:35:51 +01:00 by adam · 2 comments
Owner

Originally created by @adamweiss on GitHub (Jan 31, 2022).

NetBox version

v3.1.6

Feature type

Change to existing functionality

Proposed functionality

In order to support OIDC and SAML in social-auth-core, requirements.txt would just need to be updated with "all" extras:

social-auth-core[all]==4.1.0

This is in base_requirements.txt, but it didn't make its way over to requirements.txt.

Use case

OIDC and SAML-based SSO

Database changes

N/A

External dependencies

Change social-auth-core to social-auth-core[all] in requirements

Originally created by @adamweiss on GitHub (Jan 31, 2022). ### NetBox version v3.1.6 ### Feature type Change to existing functionality ### Proposed functionality In order to support OIDC and SAML in social-auth-core, requirements.txt would just need to be updated with "all" extras: social-auth-core[all]==4.1.0 This is in base_requirements.txt, but it didn't make its way over to requirements.txt. ### Use case OIDC and SAML-based SSO ### Database changes N/A ### External dependencies Change social-auth-core to social-auth-core[all] in requirements
adam added the status: under reviewtype: housekeeping labels 2025-12-29 19:35:52 +01:00
adam closed this issue 2025-12-29 19:35:52 +01:00
Author
Owner

@jeremystretch commented on GitHub (Feb 1, 2022):

I'm not sure we want to direct people to install these ancillary packages by default if they're not necessarily going to be used. Users always have the option of installing these in addition to the default packages.

@jeremystretch commented on GitHub (Feb 1, 2022): I'm not sure we want to direct people to install these ancillary packages by default if they're not necessarily going to be used. Users always have the option of installing these in addition to the default packages.
Author
Owner

@jeremystretch commented on GitHub (Feb 3, 2022):

Per the discussion in #8503, we're just going to drop [all] from the base requirement listing for social-auth-core due to installation issues with a dependent system library. Users can of course install additional components manually, or override this requirement in local_requirements.txt within their own installation.

@jeremystretch commented on GitHub (Feb 3, 2022): Per the discussion in #8503, we're just going to drop `[all]` from the base requirement listing for `social-auth-core` due to installation issues with a dependent system library. Users can of course install additional components manually, or override this requirement in `local_requirements.txt` within their own installation.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#6028