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

H-Captcha Support für CommonsBooking #1652

Open
datengraben opened this issue Oct 28, 2024 · 4 comments
Open

H-Captcha Support für CommonsBooking #1652

datengraben opened this issue Oct 28, 2024 · 4 comments
Labels
enhancement New feature or request good first issue php Pull requests that update Php code triage

Comments

@datengraben
Copy link
Contributor

Hcaptcha ist ein SaaS-Anbieter für die Erstellung und Verifizierung von Captcha-Bildern und -Formularen. Das hcaptcha Plugin Wordpress bietet die Integration dieser Catpcha-Abfragen in andere Plugins an. Z.B. für ultimate meber, otter blocks forms oder bbpress.

Der Buchungsprozess von CommonsBooking würde sich anbieten, um dort eine Captcha-Validierung zu intergrieren.

Auf den ersten Blick sieht es nicht wie eine große Entwicklungsaufgabe aus.
Die Integration innerhalb von BuddyPress sieht z.B. so aus: https://github.com/hCaptcha/hcaptcha-wordpress-plugin/blob/master/src/php/BuddyPress/Register.php

Voraussichtlich bräuchte es dafür keine Code-Änderung in CommonsBooking, sondern nur im hcaptcha Plugin.

@datengraben datengraben added enhancement New feature or request php Pull requests that update Php code good first issue labels Oct 28, 2024
@chriwen
Copy link
Member

chriwen commented Nov 3, 2024

@datengraben Danke für den Vorschlag. Ich frage mich jedoch, ob wir im Buchungsprozess überhaupt noch ein Captcha benötigen, da ja nur registrierte und angemeldete User überhaupt buchen können. Das Captcha müsste am besten in den Login-Prozess integriert werden - aber damit hat CB ja quasi nichts zu tun. Vielleicht könnte man es eher als Tipp auf die FAQ setzen, um den Menschen die Integration in deren bevorzugte Login- bzw. Userplugins zu erleichtern? Was meinst du? Oder hattest du einen anderen Anwendungsfall im Sinn?

@hansmorb
Copy link
Contributor

hansmorb commented Nov 3, 2024

Da stimme ich zu, ich würde das eher von Userseite aus lösen.

@datengraben
Copy link
Contributor Author

@chriwen Das stimmt. Also einen Eintrag im FAQ wäre sinnvoll! Ich habe es bei uns auch eingerichtet und wir haben z.B. die Ultimate Member Integration eingeschaltet.

Ich frage mich jedoch, ob wir im Buchungsprozess überhaupt noch ein Captcha benötigen, da ja nur registrierte und angemeldete User überhaupt buchen können.
[...]
Was meinst du? Oder hattest du einen anderen Anwendungsfall im Sinn?

@hansmorb @chriwen Die Registrierung eines Accounts kann ja unbestätigt durchgeführt werden. Das heißt die Existenz eines Wordpress-Account garantiert keinen Menschen, heißt dieser kann auch durch einen Bot gesteuert sein.

Wenn ich jetzt an eine konkrete CB Instanz denke z.B. die Flotte, wäre die Integration eines Captcha-Systems im Buchungsprozess sicher nicht uninteressant. Dort kann die Moderation (durch die Instanz-Admins) der Anmeldungen oder der Leih-Aktionen aller Accounts sicher von einer automatischen Prüfung profitieren. Ob dort Leih-Aktionen durch Bots ausgenutzt werden habe ich jetzt nicht validiert.


Da stimme ich zu, ich würde das eher von Userseite aus lösen.

@hansmorb Ich stimme dem auch zu. Das ist auch kein wirkliches Core-Thema. Die Core Software sollte es nur erlauben diese Funktionalität über Hooks einzubinden.

@hansmorb
Copy link
Contributor

Die Flotte nutzt schon SMS-Verifikation. Meinetwegen können wir einen Hook vor die Buchungsverarbeitung schalten aber ich würde da jetzt keine weitere Energie rein investieren.

@hansmorb hansmorb added the triage label Dec 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue php Pull requests that update Php code triage
Projects
Development

No branches or pull requests

3 participants