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

IBX-8817: Documented http_basic authentication config #138

Merged
merged 1 commit into from
Aug 29, 2024

Conversation

konradoboza
Copy link
Contributor

@konradoboza konradoboza commented Aug 23, 2024

🎫 Issue IBX-8817

Description:

This PR aims to document how http_basic authenticator can be used. All the necessary documentation that needs to be put in place can be found here https://symfony.com/doc/5.x/security.html#http-basic.

The example ibexa_http_basic firewall allows authorizing via browser window whenever inaccessible resource is reached by just passing regular user data.

From the technical point of view, all the work is done by built-in Symfony Symfony\Component\Security\Http\Authenticator\HttpBasicAuthenticator.

Zrzut ekranu 2024-08-23 o 12 53 52

For QA:

To properly check proposed changes one needs to uncomment ibexa_http_basic firewall and enter any URL that shouldn't be visible without being logged in using regular credentials (e.g. /admin/dashboard). After populating the browser modal with user's data, BO should become usable.

Documentation:

I suggest documenting this PR in form of a note which redirects us to the mentioned Symfony doc. Some screenshots might be also useful to better exemplify the actual feature.

@konradoboza konradoboza self-assigned this Aug 23, 2024
@konradoboza konradoboza added Doc needed The changes require some documentation Ready for review labels Aug 23, 2024
Copy link

Thanks for contribution! 🎉

To test the changes please execute:

composer config extra.symfony.endpoint https://api.github.com/repos/ibexa/recipes-dev/contents/index.json?ref=flex/pull-138

before executing the recipes.

Copy link

@konradoboza konradoboza requested a review from a team August 23, 2024 10:59
@alongosz alongosz requested a review from a team August 23, 2024 11:55
Copy link
Contributor

@tomaszszopinski tomaszszopinski left a comment

Choose a reason for hiding this comment

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

QA approved on Ibexa DXP 5.0.x commerce.

@konradoboza konradoboza merged commit f28dc8c into master Aug 29, 2024
3 checks passed
@konradoboza konradoboza deleted the ibx-8817-http-basic-usage branch August 29, 2024 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Doc needed The changes require some documentation QA approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants