• Miek Gieben's avatar
    plugin/cache: fix removing OPT (#4190) · be09f473
    Miek Gieben authored
    By checking state.Do() were are checking if the request had DO, but
    we are _always_ adding Do now - do we need to save the DO from the
    ORIGINAL request, which must be done in the ResponseWriter.
    
    Also skip OPT records in filterDNSSEC as we can't set the TTL on those
    records, this prevents writing a number to OPT's MBZ.
    
    Note none of the tests have changed and still PASS. This is due to
    the fact that CoreDNSServerAndPorts isn't a full server as we start in
    main, it lacks the scrubwriter for instance. This is not bad per se, but
    should be documented in the test code.
    Signed-off-by: default avatarMiek Gieben <miek@miek.nl>
    be09f473
handler.go 4.75 KB