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

Add auth-basic.psgi example #709

Closed
wants to merge 1 commit into from
Closed

Add auth-basic.psgi example #709

wants to merge 1 commit into from

Conversation

rwp0
Copy link

@rwp0 rwp0 commented Apr 25, 2024

This illustrates the point referenced in Plack's Auth::Basic middleware:

Using Authen::Simple::Passwd (file) to authenticate against

This illustrates the point referenced in Plack's [`Auth::Basic`][1] middleware:

Using [`Authen::Simple::Passwd`][2] (file) to authenticate against

[1]: https://metacpan.org/pod/Plack::Middleware::Auth::Basic#authenticator
[2]: https://metacpan.org/pod/Authen::Simple::Passwd
@rwp0
Copy link
Author

rwp0 commented Apr 25, 2024

Authenticator can also be an object that responds to authenticate method that takes username and password and returns boolean, so backends for Authen::Simple is perfect to use

After successfully achieving this reading the hint/reference in the documentation I wanted to save the working code somewhere (eg. in the examples) so it is kept and others could benefit from it as well.

It's OK to reject the PR (at least I tried) 🙂.

@miyagawa
Copy link
Member

I don't think anybody reads the code in eg directory. If you think the Auth::Basic documentation is lacking, I think it's much more effective to update its documentation rather than putting some random code in this directory.

@miyagawa miyagawa closed this Apr 25, 2024
@miyagawa
Copy link
Member

added the example with Authen::Simple::Passwd in 24b3867

@rwp0
Copy link
Author

rwp0 commented Apr 26, 2024

documentation

At least, I do, and those example files were very useful for me to understand how to set up a particular thing 🙂

added the example with Authen::Simple::Passwd in 24b3867

Okay, that works too, thanks!

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

Successfully merging this pull request may close these issues.

2 participants