PHP Warning: Undefined array key “REQUEST_METHOD” in RequestStorage.php

  • Resolved adamscottmartin

    (@adamscottmartin)


    1 month ago

    Have seen somewhat frequent errors with the message shown below. At least more frequently than I would like to see, but cannot replicate on demand. Looking at contents of that file it all seems normal, looks to be a very small document and I haven’t modified the php code or anything. Maybe a bug? Not sure, but posting for awareness or help to fix. Thanks.

    PHP Warning: Undefined array key “REQUEST_METHOD” in /public_html/wp-content/plugins/really-simple-ssl/core/app/Support/Helpers/Storages/RequestStorage.php on line 28

    WP version 6.9.4
    PHP version 8.3.30
    RSS plugin version 9.5.9

Viewing 9 replies - 1 through 9 (of 9 total)
  • Xoyozo

    (@xoyozo)

    1 month ago

    Hello everyone,

    I can confirm this issue:

    PHP Warning: Undefined array key “REQUEST_METHOD” in /wp-content/plugins/really-simple-ssl/core/app/Support/Helpers/Storages/RequestStorage.php on line 28

    WP version 6.9.4
    PHP version 8.4.12
    RSS version 9.5.8 -> 9.5.9

    I have a dedicated cron job running here (WP-Cron is disabled, DISABLE_WP_CRON is set to “true”), which, after updating to RSS 9.5.9, sends me an error message via email every time the cron job runs (every 5 minutes!).

    I then manually downgraded to RSS 9.5.8, and the error messages disappeared immediately. Updating back to RSS 9.5.9 immediately caused this issue to reappear.

    Right now, I find it hard to understand how such an issue could have gone unnoticed during testing (quality assurance) for a “security plugin”.

    Thanks for fixing the bug!

    Thread Starter adamscottmartin

    (@adamscottmartin)

    1 month ago

    Apologies I got busy at work. Confirmed mine also aligns with the cron schedule so this matches your experience. Appreciate you confirming the same and hopefully they can correct the bug. Cheers.

    Plugin Support Mark Amante

    (@markamante)

    1 month ago

    Hello @adamscottmartin and @xoyozo,

    Thanks for reporting this and for the detailed context 👍

    This is a PHP warning, not a critical error. It typically occurs in non-browser contexts (like cron jobs) where certain server variables (such as REQUEST_METHOD) are not available.

    We’re aware of this and have already addressed it. A fix will be included in the next release, so this warning should no longer appear.

    Thanks for flagging this!

    Xoyozo

    (@xoyozo)

    2 weeks, 3 days ago

    Hello @markamante,

    this PHP warning still appears in RSS version 9.5.10. This means the bug wasn’t fixed in “the next release”. So I’ve rolled back to RSS version 9.5.8.

    Thanks for fixing this in the next release!

    Thread Starter adamscottmartin

    (@adamscottmartin)

    2 weeks, 3 days ago

    I can confirm upgrading to latest RSS version 9.5.10 does not fix this issue on my site either. Changing status back to ‘not resolved’. Appreciate any future fixes for this bug.

    Plugin Support Jarno Vos

    (@jarnovos)

    2 weeks, 2 days ago

    Hi @adamscottmartin, @xoyozo

    Sorry, the proposed fix was not in time to make the current release; but will be part of the next one.

    You can already download the latest version of the plugin which contains the fix here. It is identical to the 9.5.10 release and only the fix for this specific warning has been added.

    Or obtain it manually from GitHub instead, here: https://github.com/Really-Simple-Plugins/really-simple-ssl/tree/fix-request-method-check

    Kind regards, Jarno

    Plugin Support Jarno Vos

    (@jarnovos)

    1 week, 2 days ago

    Hi @xoyozo @adamscottmartin,

    This fix has been included in the 9.5.10.1 update of Really Simple Security, which was just released on ww.wp.xz.cn.

    Kind regards, Jarno

    Xoyozo

    (@xoyozo)

    6 days, 4 hours ago

    Hi @jarnovos,

    Yes, I can confirm that the bug has been fixed in RSS 9.5.10.1.
    Thank you very much!

    Kind regards.

    Thread Starter adamscottmartin

    (@adamscottmartin)

    6 days, 4 hours ago

    Confirmed fixed on my site as well, thank you!!

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

You must be logged in to reply to this topic.