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

["FEAT"] Oauth validator feature flag #183

Open
Ian2012 opened this issue Jun 14, 2023 · 6 comments
Open

["FEAT"] Oauth validator feature flag #183

Ian2012 opened this issue Jun 14, 2023 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@Ian2012
Copy link
Member

Ian2012 commented Jun 14, 2023

Is your feature request related to a problem? Please describe.
In some environments, is needed to disable the extra validation for oauth tokens provided by eox-tenant:

if hasattr(settings, 'OAUTH2_PROVIDER'):
settings.OAUTH2_PROVIDER['OAUTH2_VALIDATOR_CLASS'] = 'eox_tenant.validators.EoxTenantOAuth2Validator'

Describe the solution you'd like
A feature flag USE_OAUTH_FOR_TENANTS or similar trigger mechanism

Describe alternatives you've considered

Additional context

@Ian2012 Ian2012 added the enhancement New feature or request label Jun 14, 2023
@JuanDavidBuitrago
Copy link
Member

Hi @Ian2012, thanks for your feature request

@santiagosuarezedunext Can you own this, please?

@Asespinel
Copy link
Contributor

@santiagosuarezedunext Friendly reminder to check this feature request.

@santiagosuarezedunext
Copy link

Sorry for the delay right now I'm on other projects and I can't keep track of this. I'm going to leave it in product review and when we close some issues I'll look for the author to discuss what to do with this request

@MaferMazu
Copy link
Contributor

@Ian2012, do you think this is still necessary? Or if you are not affected by this, who should be interested?

@Ian2012
Copy link
Member Author

Ian2012 commented May 24, 2024

This is still necessary, for customers who wants to configure authentication for all their sites

@Ian2012
Copy link
Member Author

Ian2012 commented May 29, 2024

This also affects the default Aspects installations

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

6 participants