Skip to content
  • Helmut Hummel's avatar
    [!!!][SECURITY] Fix link spoofing in prefixLocalAnchors · 63ae7ddd
    Helmut Hummel authored and Oliver Hader's avatar Oliver Hader committed
    Specially crafted request could lead to anchors prefixed
    with URLs to domains controlled by the attacker on the
    domain root page (home page). No other pages are affected!
    
    Fix this by prefixing the anchors with a canonical URL
    to the current request. This could lead to the situation
    that the prefix does not match the current REQUEST_URI
    which leads to a page reload instead of just "jumping" to the page section.
    
    Additionally this change assures that REQUEST_URI always starts
    with a slash, which mitigates similar attack vectors when using
    getIndpEnv('REQUEST_URI')
    
    To mitigate the impact of this breaking change, the REQUEST_URI
    is used for anchor prefix if a backend user is logged in,
    to not disturb the preview functionality of the home page.
    
    In case prefixLocalAnchors is used in the HTML parser configuration
    with prefixLocalAnchors = 2, always the canonical URL is used as prefix.
    
    This change does *not* fix, that arbitrary (non functional) GET param...
    63ae7ddd