Support antibot

Created on 19 December 2024, 4 months ago

Problem/Motivation

Antibot module will be installed by drupal_cms_starter and is configured to be used on e.g. user_login_form and also by drupal_cms_forms's contact form.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

📌 Task
Status

Active

Version

1.0

Component

Code

Created by

🇦🇹Austria arthur_lorenz Vienna

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @arthur_lorenz
  • Merge request !113Expose antibot key. → (Open) created by arthur_lorenz
  • Pipeline finished with Success
    4 months ago
    Total: 367s
    #373864
  • 🇦🇹Austria arthur_lorenz Vienna

    On Drupal side i think it's enough to expose the antibot key as a form attribute. The transformation of the key must be handled by the frontend.

  • Pipeline finished with Failed
    4 months ago
    Total: 345s
    #373885
  • Pipeline finished with Success
    4 months ago
    Total: 313s
    #373928
  • 🇸🇮Slovenia useernamee Ljubljana

    Adding the antibot token to the form attributes should work. I'd just like to check again on conditions I don't think the first one is really necessary.

  • 🇦🇹Austria arthur_lorenz Vienna

    Thx, I removed the first condition.

  • Pipeline finished with Success
    3 months ago
    Total: 410s
    #400619
  • 🇦🇹Austria fago Vienna

    lupus_decoupled_form is a generic module providing support for forms generically - it does not make sense for it to contain module specific things. That said, I wonder - https://git.drupalcode.org/project/antibot/-/blob/2.0.x/src/AntibotFormA... - is thtat data- attribute not handed over to the frontend? If so, that might be a *general* issue we should fix.

    But looking at the anitbot module file, it seems there is a bit of more stuff we need to make sure we need to support, so we might want to add a lupus_decoupled_antibot module to make sure it works.

    But the greater question is how to ship the frontend part. One way would be solving 📌 Enabling loading Drupal JavaScript via support for drupal-libraries Active what I'd love to do asap, but might take us some time to get right.

    A more straight-forward approach right now is to work-out a nice approach for having custom vue-components for drupal-cms deployed to the frontend. That will be needed for other things anyway. Then, we need this work with that, e.g. we could have some special drupal-antibot custom element which we ship that way + our integration module takes care of adding for those forms. Then the frontend component can take care of loading the Drupal antibot JS library and handling the necessary glue code.

  • 🇦🇹Austria arthur_lorenz Vienna

    I had another look at this.

    is that data- attribute not handed over to the frontend?

    The antibot key is not part of the data-attribute, it is put in the drupalSettings object only.

    I agree we should move the code current code lupus_decoupled_antibot and come up with a proper solution for FE. However, the main reason we wanted to support this was that it was part of Drupal CMS. Shortly before the release of 1.0.0 it got (temporarily?) thrown out due to a11y issues 📌 Remove Antibot module because of unaddressed accessibility issues Active

    Since there is no timeplan yet to re-add it to Drupal CMS I'll postpone this ticket.

Production build 0.71.5 2024