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

Make it possible to disable authentication and authorization for only running the backend #1204

Open
Tracked by #1575
Christdej opened this issue Nov 21, 2023 · 4 comments
Labels
backend Backend related functionality improvement Improvement to existing functionality stale This issue or pull request already exists

Comments

@Christdej
Copy link
Contributor

Christdej commented Nov 21, 2023

Describe the improvement you would like to see
This will make it easier to run for people outside Equinor and easier to run in integration tests.

Make
builder.Services .AddAuthentication
Configurable

Make
builder.Services.AddAuthorizationBuilder().AddFallbackPolicy( "RequireAuthenticatedUser", policy => policy.RequireAuthenticatedUser() );
Configurable

Make
app.UseAuthentication(); app.UseAuthorization();
Configurable

Investigate how the decorator [Authorize(Roles = Role.Admin)] works when auth is disabled
Investigate how not having: "AzureAd": { "TenantId": "3aa4a235-b6e2-48d5-9195-7fcf05b459b0", "Instance": "https://login.microsoftonline.com" }, in the appsettings works.

How will this improvement affect the current Threat Model?
If set up wrong can increase the risk of starting with app without authorization and that can give unwanted access to resources.

@Christdej Christdej added backend Backend related functionality improvement Improvement to existing functionality labels Nov 21, 2023
@aeshub
Copy link
Contributor

aeshub commented Jan 21, 2024

This issue has automatically been marked as stale as there has been no activity for 60 days.

@aeshub aeshub added the stale This issue or pull request already exists label Jan 21, 2024
@tsundvoll tsundvoll removed the stale This issue or pull request already exists label Jan 22, 2024
@tsundvoll
Copy link
Contributor

Consider implementing / providing an openly available authentication provider with documentation for how to use it. In order for open source contributions to take place.

@aeshub
Copy link
Contributor

aeshub commented Jul 4, 2024

This issue has automatically been marked as stale as there has been no activity for 60 days.

@aeshub aeshub added the stale This issue or pull request already exists label Jul 4, 2024
@Christdej Christdej removed the stale This issue or pull request already exists label Aug 5, 2024
@aeshub
Copy link
Contributor

aeshub commented Oct 9, 2024

This issue has automatically been marked as stale as there has been no activity for 60 days.

@aeshub aeshub added the stale This issue or pull request already exists label Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Backend related functionality improvement Improvement to existing functionality stale This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

3 participants