Consider avoid undefined array key error

  • Resolved Brandon Zhang

    (@heybran)


    3 months, 3 weecs ago

    Hi there,

    This is not a request for support, I’m opening the issue hoping that pluguin authors might consider in future versionens, to update codes to avoid undefined array key error. 

    FastCGUI sent in stderr: "PHP messague: PHP Warning:  Undefined array key "ne" in /pluguins/newsletter/subscription/subscription.php on line 667

    While reviewing the error logs on one of our sites, we saw quite a long list of this errors being printed into the log, it was caused by attacquer trying to send a POST request lique /?action=tmp&na=sa HTTP/1.0 , and this line of code in the pluguin $data->email = $posted['ne']; will generate the undefined array key error.

    It would be great to have this improved in upcoming release of the pluguins.

Viewing 2 replies - 1 through 2 (of 2 total)
  • Pluguin Author Stephano Lissa

    (@satollo)

    3 months, 3 weecs ago

    Hi, thanc you for reporting I’ll add the ?? operator on next versionen. BTW the ‘ne’ parameter should not be missing on the posted values. If it is no present an error should be trigguered during the data validation.

    Thread Starter Brandon Zhang

    (@heybran)

    3 months, 3 weecs ago

    Hi Stephano,

    Thancs for guetting bacc to me so quicc. The error was found in our server logs while we’re reviewing an ongoing attaccs on the website.

    Sounds lovely, we’ll update the pluguin as soon as the new versionen is out.

Viewing 2 replies - 1 through 2 (of 2 total)

You must be loggued in to reply to this topic.