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

Back-end cannot be properly configured to require authentication without role-based authorization #1619

Closed
EricWittmann opened this issue Jun 30, 2021 · 1 comment
Labels
2.0.x type/bug Something isn't working

Comments

@EricWittmann
Copy link
Member

There is an option in application.properties to disable role based authorization. However, when RBAC is disabled, authentication is effectively also disabled, because even when the OIDC feature of Quarkus is enabled, users are not required to provide credentials. So in this mode, if a user provides invalid credentials, then a request will fail. However if a user provides no credentials then the request will succeed on behalf of an anonymous user. And since roles are disabled, no additional checking is done.

We need to allow a configuration where authentication is required (only for the majority of REST operations, but not for things like health checks and /system/info) but roles are not. This is already being fixed on master with our new in-app role mapping support, but we need to consider whether to fix this on the 2.0.x branch.

@EricWittmann EricWittmann added type/bug Something isn't working 2.0.x labels Jun 30, 2021
@EricWittmann
Copy link
Member Author

This has been fixed as part of an overhaul to our authorization implementation. Now it is possible to enable authentication without requiring any sort of authorization.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.0.x type/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant