Bug report: cleantalk-common.php ~line 458

  • Resolved curtd59

    (@curtd59)


    2 years, 2 months ago

    in file: cleantalk-common.php ~line 458

    Pregmatch using @ as delimiter is unreliable, and generates endless errors, esp given how often it’s called. Add tilde before and after to render ‘safe’. Took me a while to find it. Literally flooding debug log with complaints.

    Minimum code change should be:

    pregmatch( "~@" . $exclusion . "@~", $haystack ) === 1;

    Cheers

Viewing 3 replies - 1 through 3 (of 3 total)
  • Plugin Support sergecleantalk

    (@sergecleantalk)

    Hello,

    Thank you for your report. I will ask our developers to check this.

    We will contact you within 1-3 days.

    Plugin Author alexandergull

    (@alexandergull)

    Hello @curtd59 .

    We suppose it is possible to use tildes. However:

    We tried to reproduce endless errors but got nothing. A URL regex can not be set up incorrectly because the settings page does validate it before saving.

    Can you show us a few strings that contain the errors?

    Plugin Support dimitrycleantalk

    (@dimitrycleantalk)

    Hello.
    We haven’t received your reply in a few days, so I’m going to mark this topic as “resolved”.
    If you have any further questions, you can start a new topic anytime.

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘Bug report: cleantalk-common.php ~line 458’ is closed to new replies.