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

Enable Basic Auth for testing purposes #4632

Merged
merged 21 commits into from May 16, 2024

Conversation

andreaTP
Copy link
Member

@andreaTP andreaTP commented May 3, 2024

@apicurio-bot
Copy link

apicurio-bot bot commented May 3, 2024

Thank you for creating a pull request!

Pinging @EricWittmann to respond or triage.

.basic-auth-env Outdated Show resolved Hide resolved
EricWittmann
EricWittmann previously approved these changes May 6, 2024
Copy link
Member

@EricWittmann EricWittmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - just a couple of minor comments.

@andreaTP
Copy link
Member Author

andreaTP commented May 8, 2024

This should be ready for review, you can test the final situation with a couple of pre-built container images:
Configure and run the backend using env variables:

sudo docker run --rm -it \
    -p 8080:8080 \
    -e QUARKUS_OIDC_TENANT_ENABLED=false \
    -e QUARKUS_HTTP_AUTH_BASIC=true \
    -e APICURIO_AUTH_ROLE_BASED_AUTHORIZATION=true \
    -e QUARKUS_SECURITY_USERS_EMBEDDED_PLAIN_TEXT=true \
    -e QUARKUS_SECURITY_USERS_EMBEDDED_USERS_admin=admin \
    -e QUARKUS_SECURITY_USERS_EMBEDDED_USERS_developer=developer \
    -e QUARKUS_SECURITY_USERS_EMBEDDED_USERS_readonly=readonly \
    -e QUARKUS_SECURITY_USERS_EMBEDDED_ROLES_admin=sr-admin \
    -e QUARKUS_SECURITY_USERS_EMBEDDED_ROLES_developer=sr-developer \
    -e QUARKUS_SECURITY_USERS_EMBEDDED_ROLES_readonly=sr-readonly \
    docker.io/andreatp/registry-basic-auth:latest

now you can run the UI:

docker run --rm -it -p 8888:8080 docker.io/andreatp/apicurio-registry-ui-basic-auth:latest

and now you can use the UI at http://locahost:8888

@andreaTP andreaTP marked this pull request as ready for review May 8, 2024 16:28
EricWittmann
EricWittmann previously approved these changes May 8, 2024
@paoloantinori
Copy link
Member

update: this has been validated, functionality-wise, on a Slack thread by @kevinboone

@EricWittmann EricWittmann merged commit 11b7e71 into Apicurio:main May 16, 2024
19 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants