Reduce header length when deleting cookies

  • Resolved i4nd90

    (@i4nd90)


    1 year, 8 months ago

    The server I’m working on at the moment uses Apache with Nginx reverse proxying (the Plesk default setup). For some reason Nginx gets fussy about the size of the headers being sent, and when they’re too large I get a 502 Forbidden error.

    Long story short, could the cookie blocker class perform a quick array_unique on $this->delete_cookies_list before looping through them and using setcookie to create the headers? Because of the way this variable is generated I had an array 100 items long, but only had about 5 unique values in which were being sent as headers needlessly.

    Many thanks

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Contributor jarnovos

    (@jarnovos)

    Hi @i4nd90,

    I’m assuming that the large cookie list is due to having Consent per Service enabled, and having a large number of items in LocalStorage that are being detected during the cookie scan. With an URL to your site, we could also take a closer look.

    In any case: I’ll place your request on our development backlog, so it will be considered for a future release of the plugin.

    We would also be happy to review pull requests from the open-source community for such improvements, which can be submitted via the Complianz GDPR GitHub repository: https://github.com/Really-Simple-Plugins/complianz-gdpr

    Kind regards, Jarno

    Plugin Author Rogier Lankhorst

    (@rogierlankhorst)

    @i4nd90 this is difficult to reproduce, as it needs a lot of own domain cookies on marketing level. I think the below should help:
    https://github.com/Really-Simple-Plugins/complianz-gdpr/tree/unique-cookies

    I have also limited the number of cookies to be dropped in one page load on the server side, to further limit the risk of the max header size.

    Let me know if this helps!

    Plugin Contributor jarnovos

    (@jarnovos)

    Hi @i4nd90,

    This improvement is currently included in our testing routines,and once confirmed, will be included within an upcoming version of the Complianz plugin.

    I’ll therefore mark the thread as resolved for now, feel free to let us know if you have any further questions.

    Kind regards, Jarno

    Thread Starter i4nd90

    (@i4nd90)

    Thanks so much @jarnovos and @rogierlankhors

    To be honest it’s been hard for me to replicate too, issues have been intermittent.

    When the updated version is shipped I will test on our development server and get back to you.

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘Reduce header length when deleting cookies’ is closed to new replies.