{"action":"created","issue":{"url":"https://api.github.com/repos/varnishcache/varnish-cache/issues/3169","repository_url":"https://api.github.com/repos/varnishcache/varnish-cache","labels_url":"https://api.github.com/repos/varnishcache/varnish-cache/issues/3169/labels{/name}","comments_url":"https://api.github.com/repos/varnishcache/varnish-cache/issues/3169/comments","events_url":"https://api.github.com/repos/varnishcache/varnish-cache/issues/3169/events","html_url":"https://github.com/varnishcache/varnish-cache/issues/3169","id":540448708,"node_id":"MDU6SXNzdWU1NDA0NDg3MDg=","number":3169,"title":"Handle changing Content-Encoding for inconsistent backends?","user":{"login":"Dridi","id":891744,"node_id":"MDQ6VXNlcjg5MTc0NA==","avatar_url":"https://avatars2.githubusercontent.com/u/891744?v=4","gravatar_id":"","url":"https://api.github.com/users/Dridi","html_url":"https://github.com/Dridi","followers_url":"https://api.github.com/users/Dridi/followers","following_url":"https://api.github.com/users/Dridi/following{/other_user}","gists_url":"https://api.github.com/users/Dridi/gists{/gist_id}","starred_url":"https://api.github.com/users/Dridi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Dridi/subscriptions","organizations_url":"https://api.github.com/users/Dridi/orgs","repos_url":"https://api.github.com/users/Dridi/repos","events_url":"https://api.github.com/users/Dridi/events{/privacy}","received_events_url":"https://api.github.com/users/Dridi/received_events","type":"User","site_admin":false},"labels":[{"id":506015939,"node_id":"MDU6TGFiZWw1MDYwMTU5Mzk=","url":"https://api.github.com/repos/varnishcache/varnish-cache/labels/a=invalid","name":"a=invalid","color":"404040","default":false,"description":null},{"id":357753281,"node_id":"MDU6TGFiZWwzNTc3NTMyODE=","url":"https://api.github.com/repos/varnishcache/varnish-cache/labels/c=varnishd","name":"c=varnishd","color":"3d96fb","default":false,"description":null},{"id":517445773,"node_id":"MDU6TGFiZWw1MTc0NDU3NzM=","url":"https://api.github.com/repos/varnishcache/varnish-cache/labels/r=6.0","name":"r=6.0","color":"d4c3f9","default":false,"description":null},{"id":357752916,"node_id":"MDU6TGFiZWwzNTc3NTI5MTY=","url":"https://api.github.com/repos/varnishcache/varnish-cache/labels/r=trunk","name":"r=trunk","color":"d4c5f9","default":false,"description":null}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":18,"created_at":"2019-12-19T17:20:05Z","updated_at":"2019-12-23T14:14:55Z","closed_at":null,"author_association":"MEMBER","body":"Test case by @daghf:\r\n\r\n```\r\nvarnishtest \"Ensure we don't override Content-Encoding on a cond fetch\"\r\n\r\nserver s1 {\r\n    rxreq\r\n    txresp -hdr {ETag: \"foo\"} -body {****}\r\n\r\n    rxreq\r\n    expect req.http.If-None-Match == {\"foo\"}\r\n    txresp -status 304 -hdr {ETag: W/\"foo\"} -hdr \"Content-Encoding: gzip\"\r\n\r\n    rxreq\r\n    expect req.url == \"/1\"\r\n    txresp -hdr {Etag: \"bar\"} -gzipbody {asdf}\r\n\r\n    rxreq\r\n    expect req.url == \"/1\"\r\n    expect req.http.If-None-Match == {\"bar\"}\r\n    txresp -status 304\r\n\r\n} -start\r\n\r\nvarnish v1 -vcl+backend {\r\n    sub vcl_backend_response {\r\n        set beresp.ttl = 1s;\r\n        set beresp.grace = 0s;\r\n        set beresp.keep = 1d;\r\n    }\r\n} -start\r\n\r\nclient c1 {\r\n    txreq -hdr \"Accept-Encoding: gzip\"\r\n    rxresp\r\n    expect resp.status == 200\r\n    expect resp.http.ETag == {\"foo\"}\r\n    expect resp.http.Content-Length == \"4\"\r\n    expect resp.http.Content-Encoding == <undef>\r\n\r\n    delay 1.5\r\n\r\n    txreq -hdr \"Accept-Encoding: gzip\"\r\n    rxresp\r\n    expect resp.status == 200\r\n    expect resp.http.ETag == {W/\"foo\"}\r\n    expect resp.http.Content-Length == \"4\"\r\n    expect resp.http.Content-Encoding == <undef>\r\n\r\n    # Also check that we're still OK in the general case\r\n    txreq -url \"/1\" -hdr \"Accept-Encoding: gzip\"\r\n    rxresp\r\n    expect resp.status == 200\r\n    expect resp.http.ETag == {\"bar\"}\r\n    expect resp.http.Content-Encoding == \"gzip\"\r\n\r\n    delay 1.5\r\n\r\n    txreq -url \"/1\" -hdr \"Accept-Encoding: gzip\"\r\n    rxresp\r\n    expect resp.status == 200\r\n    expect resp.http.ETag == {\"bar\"}\r\n    expect resp.http.Content-Encoding == \"gzip\"\r\n} -run\r\n```"},"comment":{"url":"https://api.github.com/repos/varnishcache/varnish-cache/issues/comments/568486570","html_url":"https://github.com/varnishcache/varnish-cache/issues/3169#issuecomment-568486570","issue_url":"https://api.github.com/repos/varnishcache/varnish-cache/issues/3169","id":568486570,"node_id":"MDEyOklzc3VlQ29tbWVudDU2ODQ4NjU3MA==","user":{"login":"nigoroll","id":1528104,"node_id":"MDQ6VXNlcjE1MjgxMDQ=","avatar_url":"https://avatars0.githubusercontent.com/u/1528104?v=4","gravatar_id":"","url":"https://api.github.com/users/nigoroll","html_url":"https://github.com/nigoroll","followers_url":"https://api.github.com/users/nigoroll/followers","following_url":"https://api.github.com/users/nigoroll/following{/other_user}","gists_url":"https://api.github.com/users/nigoroll/gists{/gist_id}","starred_url":"https://api.github.com/users/nigoroll/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nigoroll/subscriptions","organizations_url":"https://api.github.com/users/nigoroll/orgs","repos_url":"https://api.github.com/users/nigoroll/repos","events_url":"https://api.github.com/users/nigoroll/events{/privacy}","received_events_url":"https://api.github.com/users/nigoroll/received_events","type":"User","site_admin":false},"created_at":"2019-12-23T14:14:55Z","updated_at":"2019-12-23T14:14:55Z","author_association":"MEMBER","body":"bugwash: `sub vcl_backend_refresh` from #3102 would provide a convenient way for a vcl fix also.\r\n@Dridi and myself seem to agree that we want to have less policy in core, not more."}}
{"actor":{"display_login":"nigoroll"}}