Skip to content

✨ add custom selfhosted captcha #154 #259

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

Open
wants to merge 21 commits into
base: main
Choose a base branch
from

Conversation

maxlerebourg
Copy link
Owner

No description provided.

@maxlerebourg maxlerebourg self-assigned this Aug 4, 2025
@maxlerebourg maxlerebourg requested a review from mathieuHa August 6, 2025 08:59
mathieuHa
mathieuHa previously approved these changes Aug 6, 2025
Copy link
Collaborator

@mathieuHa mathieuHa left a comment

Choose a reason for hiding this comment

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

Looks good to me!
Good job @maxlerebourg

- Provider to validate the captcha, expected values are: `hcaptcha`, `recaptcha`, `turnstile` or `custom`
- CaptchaCustomJsURL
- string
- If CaptchaProvider is `custom`, URL used to load the challenge in the HTML (in case of hcaptcha: `https://hcaptcha.com/1/api.js`)
Copy link
Collaborator

@mathieuHa mathieuHa Aug 6, 2025

Choose a reason for hiding this comment

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

We could use an exemple with whicketkeeper maybe

Copy link
Owner Author

Choose a reason for hiding this comment

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

I'm waiting for @a-ve to release a new version to use the "official" image instead of the image from my fork of wicketkeeper....

@mathieuHa
Copy link
Collaborator

2025/08/11 17:29:19 ALLOWED_ORIGINS configured: [*]
2025/08/11 17:29:19 Private key file /data/wicketkeeper.key not found. Generating a new key pair.
2025/08/11 17:29:19 Failed to load or generate private key: failed to write private key to /data/wicketkeeper.key: open /data/wicketkeeper.key: permission denied

@mathieuHa
Copy link
Collaborator

One time out of 2, we get a 404 accessing localhost:8000/foo

@mathieuHa mathieuHa force-pushed the feat/Add-wicketkeeper-captcha branch from 21a715f to 4b79b1a Compare August 13, 2025 16:58
maxlerebourg and others added 9 commits August 13, 2025 19:02
* ✨ Add env for defaultStatusCode

* 📝 doc

* ✨change name of the parameter

* 🔧 Add config check

* fix lint
* Initial implementation

* fix

* fixes

* Fixes

* xx

* progress

* xx

* xx

* xx

* fix linter

* Progress

* Fixes

* xx

* xx

* Remove trace logger

* Last fix

* fix lint

* fix lint

* fix lint

---------

Co-authored-by: Max Lerebourg <maxlerebourg@gmail.com>
@mathieuHa mathieuHa force-pushed the feat/Add-wicketkeeper-captcha branch from 4b79b1a to 5b217e2 Compare August 13, 2025 17:03
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.

3 participants