There is a call to check_plain() in:

- d8/core/modules/ban/lib/Drupal/ban/EventSubscriber/BanSubscriber.php

CommentFileSizeAuthor
#3 2196805-2.patch999 bytesdivesh.kumar
#1 2196805-1.patch821 bytesdivesh.kumar
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

divesh.kumar’s picture

divesh.kumar CreditAttribution: divesh.kumar commented
Assigned: Unassigned » divesh.kumar
Status: Active » Needs review
FileSize
2196805-1.patch821 bytes

Patch attached.

thedavidmeister’s picture

thedavidmeister CreditAttribution: thedavidmeister commented
Status: Needs review » Needs work

Please use namespace aliases, see the parent issue for more details.

divesh.kumar’s picture

divesh.kumar CreditAttribution: divesh.kumar commented
Status: Needs work » Needs review
FileSize
2196805-2.patch999 bytes
thedavidmeister’s picture

thedavidmeister CreditAttribution: thedavidmeister commented
Status: Needs review » Reviewed & tested by the community

looks good to me.

JiriK’s picture

JiriK CreditAttribution: JiriK commented
Assigned: divesh.kumar » Unassigned
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.