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

Small spit-and-polish #1

Merged
merged 5 commits into from Jun 5, 2021
Merged

Small spit-and-polish #1

merged 5 commits into from Jun 5, 2021

Conversation

jwm
Copy link
Contributor

@jwm jwm commented Apr 26, 2014

Hope all y'all are well - I needed mod_auth_web for a project recently, so I put did a small spit-and-polish job on it.

@perrinjerome
Copy link

I have been using this a bit it (not against yahoo.com as AuthWebURL like in the example, but by using a dedicated API endpoint from a custom-made web application) and it seems working fine.

Any reason this was not merged ?

/cc @Castaglia

@Castaglia Castaglia merged commit 871e21c into proftpd:master Jun 5, 2021
@Castaglia
Copy link
Member

Thanks! I wanted to add GitHub Actions support for CI builds (just done) before merging this.

@perrinjerome
Copy link

Thank you ! I can also confirm that after this being merged, it works fine for me. I was using with proftpd 1.3.7a and this plus the recent commits on master branch of https://github.com/proftpd/mod_auth_web

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.

None yet

3 participants