Details

    • Type: Problem report
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 3.4.11
    • Component/s: Frontend (F)
    • Labels:
      None
    • Environment:
      Apache2, Debian 9.4, official Debian repos.
    • Team:
      Team B
    • Sprint:
      Sprint 46, Nov 2018, Sprint 47, Dec 2018
    • Story Points:
      0.125

      Description

      Excuse me if I'm a little bit too harsh, I spent hours debugging this.

      This is about /include/page_header.php, line 134 to 137 on 3.4.11-1+stretch. Also have a look at ZBX-13133

      On one hand it is useful to set the X-Frame-Options header, as this is application specific, and only the application developers know if the application has certain constraints and would not work if the header is set. On the other hand, HSTS is about the capabilities of the web server software, and not about the web application.

      Therefore HSTS is something that should never be done at application level. You also don't configure TLS certificates in Zabbix, but in your apache, nginx, whatever. This causes the following problems:

      1) If your https is misconfigured, or has a self-signed certificate you effectively locked yourself out of your monitoring. 

      2) If you configured your server to provide HSTS headers, you get two and maybe even conflicting HSTS headers in your HTTP response, which violates RFC 6797, or makes you write exceptions for Zabbix in your httpd config.

      3) This is hard to debug, as there seems to be no documentation of this behaviour at all.

      4) It statically sets the max-age to 31557600 seconds, which is also not configurable.

       

      Therefore HSTS headers should never be added by anyone but the web server. If you insist on Zabbix being able to add HSTS headers, you should seriously leave it disable as default, make it configurable, and document it.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Miks.Kronkalns Miks Kronkalns
                Reporter:
                maxried@posteo.de Max Ried
              • Votes:
                1 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: