Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chx created an issue. See original summary.

dawehner’s picture

  1. +++ b/docroot/modules/contrib/mollom/mollom.module
    @@ -341,6 +343,48 @@ function user_mollom_form_list() {
    +      'node-report' => [
    

    I would have used 'entity-report' or so

  2. +++ b/docroot/modules/contrib/mollom/mollom.routing.yml
    @@ -85,3 +85,14 @@ mollom.blacklist.delete:
    +  requirements:
    +    _csrf_token: 'TRUE'
    

    I try to understand why someone needs a CSRF token for a form. Isn't the point that this form is using a POST anyway, so FAPI should protect that automatically?

dawehner’s picture

Issue summary: View changes
chx’s picture

Issue summary: View changes

The last submitted patch, 2: 2778209_2.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 4: 2778209_4.patch, failed testing.

eshta’s picture

Status: Needs work » Needs review
FileSize
4.75 KB

Thanks for all the work on this. Re-rolling now that Mollom's test suite is fixed. Go bots!