8000 Replace hCaptcha with Altcha #235 by tdruez · Pull Request #278 · aboutcode-org/dejacode · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Replace hCaptcha with Altcha #235 #278

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 18 commits into from
Mar 31, 2025
Merged

Replace hCaptcha with Altcha #235 #278

merged 18 commits into from
Mar 31, 2025

Conversation

tdruez
Copy link
Contributor
@tdruez tdruez commented Mar 7, 2025

No description provided.

@tdruez
Replace hCaptcha with Altcha #235
38e5496 
Signed-off-by: tdruez <tdruez@nexb.com>
@pombredanne
Copy link
Member

@tdruez 👍

  • Can we / should we self host the server?
  • What could we do to also deploy the same to SCIO, VCIO and FederatedCode?
  • You are missing an about file for the library

@tdruez
Copy link
Contributor Author
tdruez commented Mar 10, 2025 •
edited
Loading

Can we / should we self host the server?

Yes, this implementation is self-hosted:

There are no dependencies on an external server and no need for extra accounts or API keys.

What could we do to also deploy the same to SCIO, VCIO and FederatedCode?

I'm thinking about packaging this PR content (field and view), once stable, as a Django library, similar to the https://pypi.org/project/django-hcaptcha-field/ we've been using until now.

The implementation in our various apps would be quite simple:

  1. Add the AltchaField to your (registration) form:
from django_altcha import AltchaField

class RegistrationForm(forms.Form):
    altcha_token = AltchaField()
  1. Add the challenge view to your URLs (only required for the self-hosted mode, the alternative is to provide settings to the external server, URL and API key)
from django_altcha import get_altcha_challenge_view

urlpatterns += [
    path("altcha/", get_altcha_challenge_view, name="get_altcha_challenge"),
]

This should probably be released under MIT to match the current license on the Python lib https://github.com/altcha-org/altcha-lib-py and the JS "Web Component" https://github.com/altcha-org/altcha included in the HTML rendered page.

You are missing an about file for the library

Yes, this PR is still in Prototype mode, making sure this approach will work end to end first.
I usually add the .ABOUT files once the set of new libraries, and their versions, are pinned down.

@pombredanne Let me know your take on releasing this as a library as suggested above.

@tdruez tdruez marked this pull request as draft March 10, 2025 04:02
tdruez added 14 commits March 10, 2025 17:49
@tdruez
Refine the Altcha implementation #235
f2547a2 
Signed-off-by: tdruez <tdruez@nexb.com>
@tdruez
Refactor the widget to provide option on the Field instance #235
093d500 
Signed-off-by: tdruez <tdruez@nexb.com>
@tdruez
Add all options to the AltchaWidget #235
2fdec19 
Signed-off-by: tdruez <tdruez@nexb.com>
@tdruez
Remove the need for a dedicated view #235
5560103 
Signed-off-by: tdruez <tdruez@nexb.com>
@tdruez
Move the altcha code to its own module #235
ed86e34 
Signed-off-by: tdruez <tdruez@nexb.com>
@tdruez
Add the altcha.min.js as a local static file #235
e6beab8 
Signed-off-by: tdruez <tdruez@nexb.com>
@tdruez
Remove debug instructions #235
a5ff954 
Signed-off-by: tdruez <tdruez@nexb.com>
@tdruez
Move the generate challenge at the widget level #235
bf9626d 
Signed-off-by: tdruez <tdruez@nexb.com>
@tdruez
Fix failing tests #235
5376849 
Signed-off-by: tdruez <tdruez@nexb.com>
@tdruez
Merge branch 'main' into 235-altcha
f873c76
@tdruez
Upgrade the Altcha JS lib to latest version 1.4.0 #235
3d7dea0 
Signed-off-by: tdruez <tdruez@nexb.com>
@tdruez
Replace the local django_altcha module by wheel dist #235
5e8d54e 
Signed-off-by: tdruez <tdruez@nexb.com>
@tdruez
Merge branch 'main' into 235-altcha
72d99e8
@tdruez
Upgrade django-altcha to latest release #235
947d2dd 
Signed-off-by: tdruez <tdruez@nexb.com>
@tdruez tdruez marked this pull request as ready for review March 31, 2025 10:57
tdruez added 3 commits March 31, 2025 15:12
@tdruez
Upgrade Django and Jinja2 to latest release #235
5dcddfd 
Signed-off-by: tdruez <tdruez@nexb.com>
@tdruez
Upgrade djgnao 9E22 _altcha to latest version #235
e0fe94b 
Signed-off-by: tdruez <tdruez@nexb.com>
@tdruez
Fix failing tests #235
ec632c5 
Signed-off-by: tdruez <tdruez@nexb.com>
@tdruez tdruez merged commit 35ad746 into main Mar 31, 2025
4 checks passed
@tdruez tdruez deleted the 235-altcha branch March 31, 2025 11:57
@tdruez tdruez mentioned this pull request Mar 31, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tdruez @pombredanne
0