| 1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374757677787980818283848586878889909192939495969798991001011021031041051061071081091101111121131141151161171181191201211221231241251261271281291301311321331341351361371381391401411421431441451461471481491501511521531541551561571581591601611621631641651661671681691701711721731741751761771781791801811821831841851861871881891901911921931941951961971981992002012022032042052062072082092102112122132142152162172182192202212222232242252262272282292302312322332342352362372382392402412422432442452462472482492502512522532542552562572582592602612622632642652662672682692702712722732742752762772782792802812822832842852862872882892902912922932942952962972982993003013023033043053063073083093103113123133143153163173183193203213223233243253263273283293303313323333343353363373383393403413423433443453463473483493503513523533543553563573583593603613623633643653663673683693703713723733743753763773783793803813823833843853863873883893903913923933943953963973983994004014024034044054064074084094104114124134144154164174184194204214224234244254264274284294304314324334344354364374384394404414424434444454464474484494504514524534544554564574584594604614624634644654664674684694704714724734744754764774784794804814824834844854864874884894904914924934944954964974984995005015025035045055065075085095105115125135145155165175185195205215225235245255265275285295305315325335345355365375385395405415425435445455465475485495505515525535545555565575585595605615625635645655665675685695705715725735745755765775785795805815825835845855865875885895905915925935945955965975985996006016026036046056066076086096106116126136146156166176186196206216226236246256266276286296306316326336346356366376386396406416426436446456466476486496506516526536546556566576586596606616626636646656666676686696706716726736746756766776786796806816826836846856866876886896906916926936946956966976986997007017027037047057067077087097107117127137147157167177187197207217227237247257267277287297307317327337347357367377387397407417427437447457467477487497507517527537547557567577587597607617627637647657667677687697707717727737747757767777787797807817827837847857867877887897907917927937947957967977987998008018028038048058068078088098108118128138148158168178188198208218228238248258268278288298308318328338348358368378388398408418428438448458468478488498508518528538548558568578588598608618628638648658668678688698708718728738748758768778788798808818828838848858868878888898908918928938948958968978988999009019029039049059069079089099109119129139149159169179189199209219229239249259269279289299309319329339349359369379389399409419429439449459469479489499509519529539549559569579589599609619629639649659669679689699709719729739749759769779789799809819829839849859869879889899909919929939949959969979989991000100110021003100410051006100710081009101010111012101310141015101610171018101910201021102210231024102510261027102810291030103110321033103410351036103710381039104010411042104310441045104610471048104910501051105210531054105510561057105810591060106110621063106410651066106710681069107010711072107310741075107610771078107910801081108210831084108510861087108810891090109110921093109410951096109710981099110011011102110311041105110611071108110911101111111211131114111511161117111811191120112111221123112411251126112711281129113011311132113311341135113611371138113911401141114211431144114511461147114811491150115111521153115411551156115711581159116011611162116311641165116611671168116911701171117211731174117511761177117811791180118111821183118411851186118711881189119011911192119311941195119611971198119912001201120212031204120512061207120812091210121112121213121412151216121712181219122012211222122312241225122612271228122912301231123212331234123512361237123812391240124112421243124412451246124712481249125012511252125312541255125612571258125912601261126212631264126512661267126812691270127112721273127412751276127712781279128012811282128312841285128612871288128912901291129212931294129512961297129812991300130113021303130413051306130713081309131013111312131313141315131613171318131913201321132213231324132513261327132813291330133113321333133413351336133713381339134013411342134313441345134613471348134913501351135213531354135513561357135813591360 |
- package httpcache
- import (
- "bytes"
- "errors"
- "flag"
- "io"
- "io/ioutil"
- "net/http"
- "net/http/httptest"
- "os"
- "strconv"
- "testing"
- "time"
- )
- var s struct {
- server *httptest.Server
- client http.Client
- transport *Transport
- done chan struct{} // Closed to unlock infinite handlers.
- }
- type fakeClock struct {
- elapsed time.Duration
- }
- func (c *fakeClock) since(t time.Time) time.Duration {
- return c.elapsed
- }
- func TestMain(m *testing.M) {
- flag.Parse()
- setup()
- code := m.Run()
- teardown()
- os.Exit(code)
- }
- func setup() {
- tp := NewMemoryCacheTransport()
- client := http.Client{Transport: tp}
- s.transport = tp
- s.client = client
- s.done = make(chan struct{})
- mux := http.NewServeMux()
- s.server = httptest.NewServer(mux)
- mux.HandleFunc("/", http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- w.Header().Set("Cache-Control", "max-age=3600")
- }))
- mux.HandleFunc("/method", http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- w.Header().Set("Cache-Control", "max-age=3600")
- w.Write([]byte(r.Method))
- }))
- mux.HandleFunc("/range", http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- lm := "Fri, 14 Dec 2010 01:01:50 GMT"
- if r.Header.Get("if-modified-since") == lm {
- w.WriteHeader(http.StatusNotModified)
- return
- }
- w.Header().Set("last-modified", lm)
- if r.Header.Get("range") == "bytes=4-9" {
- w.WriteHeader(http.StatusPartialContent)
- w.Write([]byte(" text "))
- return
- }
- w.Write([]byte("Some text content"))
- }))
- mux.HandleFunc("/nostore", http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- w.Header().Set("Cache-Control", "no-store")
- }))
- mux.HandleFunc("/etag", http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- etag := "124567"
- if r.Header.Get("if-none-match") == etag {
- w.WriteHeader(http.StatusNotModified)
- return
- }
- w.Header().Set("etag", etag)
- }))
- mux.HandleFunc("/lastmodified", http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- lm := "Fri, 14 Dec 2010 01:01:50 GMT"
- if r.Header.Get("if-modified-since") == lm {
- w.WriteHeader(http.StatusNotModified)
- return
- }
- w.Header().Set("last-modified", lm)
- }))
- mux.HandleFunc("/varyaccept", http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- w.Header().Set("Cache-Control", "max-age=3600")
- w.Header().Set("Content-Type", "text/plain")
- w.Header().Set("Vary", "Accept")
- w.Write([]byte("Some text content"))
- }))
- mux.HandleFunc("/doublevary", http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- w.Header().Set("Cache-Control", "max-age=3600")
- w.Header().Set("Content-Type", "text/plain")
- w.Header().Set("Vary", "Accept, Accept-Language")
- w.Write([]byte("Some text content"))
- }))
- mux.HandleFunc("/2varyheaders", http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- w.Header().Set("Cache-Control", "max-age=3600")
- w.Header().Set("Content-Type", "text/plain")
- w.Header().Add("Vary", "Accept")
- w.Header().Add("Vary", "Accept-Language")
- w.Write([]byte("Some text content"))
- }))
- mux.HandleFunc("/varyunused", http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- w.Header().Set("Cache-Control", "max-age=3600")
- w.Header().Set("Content-Type", "text/plain")
- w.Header().Set("Vary", "X-Madeup-Header")
- w.Write([]byte("Some text content"))
- }))
- updateFieldsCounter := 0
- mux.HandleFunc("/updatefields", http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- w.Header().Set("X-Counter", strconv.Itoa(updateFieldsCounter))
- w.Header().Set("Etag", `"e"`)
- updateFieldsCounter++
- if r.Header.Get("if-none-match") != "" {
- w.WriteHeader(http.StatusNotModified)
- return
- }
- w.Write([]byte("Some text content"))
- }))
- // Take 3 seconds to return 200 OK (for testing client timeouts).
- mux.HandleFunc("/3seconds", http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- time.Sleep(3 * time.Second)
- }))
- mux.HandleFunc("/infinite", http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- for {
- select {
- case <-s.done:
- return
- default:
- w.Write([]byte{0})
- }
- }
- }))
- }
- func teardown() {
- close(s.done)
- s.server.Close()
- }
- func resetTest() {
- s.transport.Cache = NewMemoryCache()
- clock = &realClock{}
- }
- // TestCacheableMethod ensures that uncacheable method does not get stored
- // in cache and get incorrectly used for a following cacheable method request.
- func TestCacheableMethod(t *testing.T) {
- resetTest()
- {
- req, err := http.NewRequest("POST", s.server.URL+"/method", nil)
- if err != nil {
- t.Fatal(err)
- }
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- var buf bytes.Buffer
- _, err = io.Copy(&buf, resp.Body)
- if err != nil {
- t.Fatal(err)
- }
- err = resp.Body.Close()
- if err != nil {
- t.Fatal(err)
- }
- if got, want := buf.String(), "POST"; got != want {
- t.Errorf("got %q, want %q", got, want)
- }
- if resp.StatusCode != http.StatusOK {
- t.Errorf("response status code isn't 200 OK: %v", resp.StatusCode)
- }
- }
- {
- req, err := http.NewRequest("GET", s.server.URL+"/method", nil)
- if err != nil {
- t.Fatal(err)
- }
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- var buf bytes.Buffer
- _, err = io.Copy(&buf, resp.Body)
- if err != nil {
- t.Fatal(err)
- }
- err = resp.Body.Close()
- if err != nil {
- t.Fatal(err)
- }
- if got, want := buf.String(), "GET"; got != want {
- t.Errorf("got wrong body %q, want %q", got, want)
- }
- if resp.StatusCode != http.StatusOK {
- t.Errorf("response status code isn't 200 OK: %v", resp.StatusCode)
- }
- if resp.Header.Get(XFromCache) != "" {
- t.Errorf("XFromCache header isn't blank")
- }
- }
- }
- func TestDontStorePartialRangeInCache(t *testing.T) {
- resetTest()
- {
- req, err := http.NewRequest("GET", s.server.URL+"/range", nil)
- if err != nil {
- t.Fatal(err)
- }
- req.Header.Set("range", "bytes=4-9")
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- var buf bytes.Buffer
- _, err = io.Copy(&buf, resp.Body)
- if err != nil {
- t.Fatal(err)
- }
- err = resp.Body.Close()
- if err != nil {
- t.Fatal(err)
- }
- if got, want := buf.String(), " text "; got != want {
- t.Errorf("got %q, want %q", got, want)
- }
- if resp.StatusCode != http.StatusPartialContent {
- t.Errorf("response status code isn't 206 Partial Content: %v", resp.StatusCode)
- }
- }
- {
- req, err := http.NewRequest("GET", s.server.URL+"/range", nil)
- if err != nil {
- t.Fatal(err)
- }
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- var buf bytes.Buffer
- _, err = io.Copy(&buf, resp.Body)
- if err != nil {
- t.Fatal(err)
- }
- err = resp.Body.Close()
- if err != nil {
- t.Fatal(err)
- }
- if got, want := buf.String(), "Some text content"; got != want {
- t.Errorf("got %q, want %q", got, want)
- }
- if resp.StatusCode != http.StatusOK {
- t.Errorf("response status code isn't 200 OK: %v", resp.StatusCode)
- }
- if resp.Header.Get(XFromCache) != "" {
- t.Error("XFromCache header isn't blank")
- }
- }
- {
- req, err := http.NewRequest("GET", s.server.URL+"/range", nil)
- if err != nil {
- t.Fatal(err)
- }
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- var buf bytes.Buffer
- _, err = io.Copy(&buf, resp.Body)
- if err != nil {
- t.Fatal(err)
- }
- err = resp.Body.Close()
- if err != nil {
- t.Fatal(err)
- }
- if got, want := buf.String(), "Some text content"; got != want {
- t.Errorf("got %q, want %q", got, want)
- }
- if resp.StatusCode != http.StatusOK {
- t.Errorf("response status code isn't 200 OK: %v", resp.StatusCode)
- }
- if resp.Header.Get(XFromCache) != "1" {
- t.Errorf(`XFromCache header isn't "1": %v`, resp.Header.Get(XFromCache))
- }
- }
- {
- req, err := http.NewRequest("GET", s.server.URL+"/range", nil)
- if err != nil {
- t.Fatal(err)
- }
- req.Header.Set("range", "bytes=4-9")
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- var buf bytes.Buffer
- _, err = io.Copy(&buf, resp.Body)
- if err != nil {
- t.Fatal(err)
- }
- err = resp.Body.Close()
- if err != nil {
- t.Fatal(err)
- }
- if got, want := buf.String(), " text "; got != want {
- t.Errorf("got %q, want %q", got, want)
- }
- if resp.StatusCode != http.StatusPartialContent {
- t.Errorf("response status code isn't 206 Partial Content: %v", resp.StatusCode)
- }
- }
- }
- func TestCacheOnlyIfBodyRead(t *testing.T) {
- resetTest()
- {
- req, err := http.NewRequest("GET", s.server.URL, nil)
- if err != nil {
- t.Fatal(err)
- }
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- if resp.Header.Get(XFromCache) != "" {
- t.Fatal("XFromCache header isn't blank")
- }
- // We do not read the body
- resp.Body.Close()
- }
- {
- req, err := http.NewRequest("GET", s.server.URL, nil)
- if err != nil {
- t.Fatal(err)
- }
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "" {
- t.Fatalf("XFromCache header isn't blank")
- }
- }
- }
- func TestOnlyReadBodyOnDemand(t *testing.T) {
- resetTest()
- req, err := http.NewRequest("GET", s.server.URL+"/infinite", nil)
- if err != nil {
- t.Fatal(err)
- }
- resp, err := s.client.Do(req) // This shouldn't hang forever.
- if err != nil {
- t.Fatal(err)
- }
- buf := make([]byte, 10) // Only partially read the body.
- _, err = resp.Body.Read(buf)
- if err != nil {
- t.Fatal(err)
- }
- resp.Body.Close()
- }
- func TestGetOnlyIfCachedHit(t *testing.T) {
- resetTest()
- {
- req, err := http.NewRequest("GET", s.server.URL, nil)
- if err != nil {
- t.Fatal(err)
- }
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "" {
- t.Fatal("XFromCache header isn't blank")
- }
- _, err = ioutil.ReadAll(resp.Body)
- if err != nil {
- t.Fatal(err)
- }
- }
- {
- req, err := http.NewRequest("GET", s.server.URL, nil)
- if err != nil {
- t.Fatal(err)
- }
- req.Header.Add("cache-control", "only-if-cached")
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "1" {
- t.Fatalf(`XFromCache header isn't "1": %v`, resp.Header.Get(XFromCache))
- }
- if resp.StatusCode != http.StatusOK {
- t.Fatalf("response status code isn't 200 OK: %v", resp.StatusCode)
- }
- }
- }
- func TestGetOnlyIfCachedMiss(t *testing.T) {
- resetTest()
- req, err := http.NewRequest("GET", s.server.URL, nil)
- if err != nil {
- t.Fatal(err)
- }
- req.Header.Add("cache-control", "only-if-cached")
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "" {
- t.Fatal("XFromCache header isn't blank")
- }
- if resp.StatusCode != http.StatusGatewayTimeout {
- t.Fatalf("response status code isn't 504 GatewayTimeout: %v", resp.StatusCode)
- }
- }
- func TestGetNoStoreRequest(t *testing.T) {
- resetTest()
- req, err := http.NewRequest("GET", s.server.URL, nil)
- if err != nil {
- t.Fatal(err)
- }
- req.Header.Add("Cache-Control", "no-store")
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "" {
- t.Fatal("XFromCache header isn't blank")
- }
- }
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "" {
- t.Fatal("XFromCache header isn't blank")
- }
- }
- }
- func TestGetNoStoreResponse(t *testing.T) {
- resetTest()
- req, err := http.NewRequest("GET", s.server.URL+"/nostore", nil)
- if err != nil {
- t.Fatal(err)
- }
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "" {
- t.Fatal("XFromCache header isn't blank")
- }
- }
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "" {
- t.Fatal("XFromCache header isn't blank")
- }
- }
- }
- func TestGetWithEtag(t *testing.T) {
- resetTest()
- req, err := http.NewRequest("GET", s.server.URL+"/etag", nil)
- if err != nil {
- t.Fatal(err)
- }
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "" {
- t.Fatal("XFromCache header isn't blank")
- }
- _, err = ioutil.ReadAll(resp.Body)
- if err != nil {
- t.Fatal(err)
- }
- }
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "1" {
- t.Fatalf(`XFromCache header isn't "1": %v`, resp.Header.Get(XFromCache))
- }
- // additional assertions to verify that 304 response is converted properly
- if resp.StatusCode != http.StatusOK {
- t.Fatalf("response status code isn't 200 OK: %v", resp.StatusCode)
- }
- if _, ok := resp.Header["Connection"]; ok {
- t.Fatalf("Connection header isn't absent")
- }
- }
- }
- func TestGetWithLastModified(t *testing.T) {
- resetTest()
- req, err := http.NewRequest("GET", s.server.URL+"/lastmodified", nil)
- if err != nil {
- t.Fatal(err)
- }
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "" {
- t.Fatal("XFromCache header isn't blank")
- }
- _, err = ioutil.ReadAll(resp.Body)
- if err != nil {
- t.Fatal(err)
- }
- }
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "1" {
- t.Fatalf(`XFromCache header isn't "1": %v`, resp.Header.Get(XFromCache))
- }
- }
- }
- func TestGetWithVary(t *testing.T) {
- resetTest()
- req, err := http.NewRequest("GET", s.server.URL+"/varyaccept", nil)
- if err != nil {
- t.Fatal(err)
- }
- req.Header.Set("Accept", "text/plain")
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get("Vary") != "Accept" {
- t.Fatalf(`Vary header isn't "Accept": %v`, resp.Header.Get("Vary"))
- }
- _, err = ioutil.ReadAll(resp.Body)
- if err != nil {
- t.Fatal(err)
- }
- }
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "1" {
- t.Fatalf(`XFromCache header isn't "1": %v`, resp.Header.Get(XFromCache))
- }
- }
- req.Header.Set("Accept", "text/html")
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "" {
- t.Fatal("XFromCache header isn't blank")
- }
- }
- req.Header.Set("Accept", "")
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "" {
- t.Fatal("XFromCache header isn't blank")
- }
- }
- }
- func TestGetWithDoubleVary(t *testing.T) {
- resetTest()
- req, err := http.NewRequest("GET", s.server.URL+"/doublevary", nil)
- if err != nil {
- t.Fatal(err)
- }
- req.Header.Set("Accept", "text/plain")
- req.Header.Set("Accept-Language", "da, en-gb;q=0.8, en;q=0.7")
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get("Vary") == "" {
- t.Fatalf(`Vary header is blank`)
- }
- _, err = ioutil.ReadAll(resp.Body)
- if err != nil {
- t.Fatal(err)
- }
- }
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "1" {
- t.Fatalf(`XFromCache header isn't "1": %v`, resp.Header.Get(XFromCache))
- }
- }
- req.Header.Set("Accept-Language", "")
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "" {
- t.Fatal("XFromCache header isn't blank")
- }
- }
- req.Header.Set("Accept-Language", "da")
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "" {
- t.Fatal("XFromCache header isn't blank")
- }
- }
- }
- func TestGetWith2VaryHeaders(t *testing.T) {
- resetTest()
- // Tests that multiple Vary headers' comma-separated lists are
- // merged. See https://github.com/gregjones/httpcache/issues/27.
- const (
- accept = "text/plain"
- acceptLanguage = "da, en-gb;q=0.8, en;q=0.7"
- )
- req, err := http.NewRequest("GET", s.server.URL+"/2varyheaders", nil)
- if err != nil {
- t.Fatal(err)
- }
- req.Header.Set("Accept", accept)
- req.Header.Set("Accept-Language", acceptLanguage)
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get("Vary") == "" {
- t.Fatalf(`Vary header is blank`)
- }
- _, err = ioutil.ReadAll(resp.Body)
- if err != nil {
- t.Fatal(err)
- }
- }
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "1" {
- t.Fatalf(`XFromCache header isn't "1": %v`, resp.Header.Get(XFromCache))
- }
- }
- req.Header.Set("Accept-Language", "")
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "" {
- t.Fatal("XFromCache header isn't blank")
- }
- }
- req.Header.Set("Accept-Language", "da")
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "" {
- t.Fatal("XFromCache header isn't blank")
- }
- }
- req.Header.Set("Accept-Language", acceptLanguage)
- req.Header.Set("Accept", "")
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "" {
- t.Fatal("XFromCache header isn't blank")
- }
- }
- req.Header.Set("Accept", "image/png")
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "" {
- t.Fatal("XFromCache header isn't blank")
- }
- _, err = ioutil.ReadAll(resp.Body)
- if err != nil {
- t.Fatal(err)
- }
- }
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "1" {
- t.Fatalf(`XFromCache header isn't "1": %v`, resp.Header.Get(XFromCache))
- }
- }
- }
- func TestGetVaryUnused(t *testing.T) {
- resetTest()
- req, err := http.NewRequest("GET", s.server.URL+"/varyunused", nil)
- if err != nil {
- t.Fatal(err)
- }
- req.Header.Set("Accept", "text/plain")
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get("Vary") == "" {
- t.Fatalf(`Vary header is blank`)
- }
- _, err = ioutil.ReadAll(resp.Body)
- if err != nil {
- t.Fatal(err)
- }
- }
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "1" {
- t.Fatalf(`XFromCache header isn't "1": %v`, resp.Header.Get(XFromCache))
- }
- }
- }
- func TestUpdateFields(t *testing.T) {
- resetTest()
- req, err := http.NewRequest("GET", s.server.URL+"/updatefields", nil)
- if err != nil {
- t.Fatal(err)
- }
- var counter, counter2 string
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- counter = resp.Header.Get("x-counter")
- _, err = ioutil.ReadAll(resp.Body)
- if err != nil {
- t.Fatal(err)
- }
- }
- {
- resp, err := s.client.Do(req)
- if err != nil {
- t.Fatal(err)
- }
- defer resp.Body.Close()
- if resp.Header.Get(XFromCache) != "1" {
- t.Fatalf(`XFromCache header isn't "1": %v`, resp.Header.Get(XFromCache))
- }
- counter2 = resp.Header.Get("x-counter")
- }
- if counter == counter2 {
- t.Fatalf(`both "x-counter" values are equal: %v %v`, counter, counter2)
- }
- }
- func TestParseCacheControl(t *testing.T) {
- resetTest()
- h := http.Header{}
- for range parseCacheControl(h) {
- t.Fatal("cacheControl should be empty")
- }
- h.Set("cache-control", "no-cache")
- {
- cc := parseCacheControl(h)
- if _, ok := cc["foo"]; ok {
- t.Error(`Value "foo" shouldn't exist`)
- }
- noCache, ok := cc["no-cache"]
- if !ok {
- t.Fatalf(`"no-cache" value isn't set`)
- }
- if noCache != "" {
- t.Fatalf(`"no-cache" value isn't blank: %v`, noCache)
- }
- }
- h.Set("cache-control", "no-cache, max-age=3600")
- {
- cc := parseCacheControl(h)
- noCache, ok := cc["no-cache"]
- if !ok {
- t.Fatalf(`"no-cache" value isn't set`)
- }
- if noCache != "" {
- t.Fatalf(`"no-cache" value isn't blank: %v`, noCache)
- }
- if cc["max-age"] != "3600" {
- t.Fatalf(`"max-age" value isn't "3600": %v`, cc["max-age"])
- }
- }
- }
- func TestNoCacheRequestExpiration(t *testing.T) {
- resetTest()
- respHeaders := http.Header{}
- respHeaders.Set("Cache-Control", "max-age=7200")
- reqHeaders := http.Header{}
- reqHeaders.Set("Cache-Control", "no-cache")
- if getFreshness(respHeaders, reqHeaders) != transparent {
- t.Fatal("freshness isn't transparent")
- }
- }
- func TestNoCacheResponseExpiration(t *testing.T) {
- resetTest()
- respHeaders := http.Header{}
- respHeaders.Set("Cache-Control", "no-cache")
- respHeaders.Set("Expires", "Wed, 19 Apr 3000 11:43:00 GMT")
- reqHeaders := http.Header{}
- if getFreshness(respHeaders, reqHeaders) != stale {
- t.Fatal("freshness isn't stale")
- }
- }
- func TestReqMustRevalidate(t *testing.T) {
- resetTest()
- // not paying attention to request setting max-stale means never returning stale
- // responses, so always acting as if must-revalidate is set
- respHeaders := http.Header{}
- reqHeaders := http.Header{}
- reqHeaders.Set("Cache-Control", "must-revalidate")
- if getFreshness(respHeaders, reqHeaders) != stale {
- t.Fatal("freshness isn't stale")
- }
- }
- func TestRespMustRevalidate(t *testing.T) {
- resetTest()
- respHeaders := http.Header{}
- respHeaders.Set("Cache-Control", "must-revalidate")
- reqHeaders := http.Header{}
- if getFreshness(respHeaders, reqHeaders) != stale {
- t.Fatal("freshness isn't stale")
- }
- }
- func TestFreshExpiration(t *testing.T) {
- resetTest()
- now := time.Now()
- respHeaders := http.Header{}
- respHeaders.Set("date", now.Format(time.RFC1123))
- respHeaders.Set("expires", now.Add(time.Duration(2)*time.Second).Format(time.RFC1123))
- reqHeaders := http.Header{}
- if getFreshness(respHeaders, reqHeaders) != fresh {
- t.Fatal("freshness isn't fresh")
- }
- clock = &fakeClock{elapsed: 3 * time.Second}
- if getFreshness(respHeaders, reqHeaders) != stale {
- t.Fatal("freshness isn't stale")
- }
- }
- func TestMaxAge(t *testing.T) {
- resetTest()
- now := time.Now()
- respHeaders := http.Header{}
- respHeaders.Set("date", now.Format(time.RFC1123))
- respHeaders.Set("cache-control", "max-age=2")
- reqHeaders := http.Header{}
- if getFreshness(respHeaders, reqHeaders) != fresh {
- t.Fatal("freshness isn't fresh")
- }
- clock = &fakeClock{elapsed: 3 * time.Second}
- if getFreshness(respHeaders, reqHeaders) != stale {
- t.Fatal("freshness isn't stale")
- }
- }
- func TestMaxAgeZero(t *testing.T) {
- resetTest()
- now := time.Now()
- respHeaders := http.Header{}
- respHeaders.Set("date", now.Format(time.RFC1123))
- respHeaders.Set("cache-control", "max-age=0")
- reqHeaders := http.Header{}
- if getFreshness(respHeaders, reqHeaders) != stale {
- t.Fatal("freshness isn't stale")
- }
- }
- func TestBothMaxAge(t *testing.T) {
- resetTest()
- now := time.Now()
- respHeaders := http.Header{}
- respHeaders.Set("date", now.Format(time.RFC1123))
- respHeaders.Set("cache-control", "max-age=2")
- reqHeaders := http.Header{}
- reqHeaders.Set("cache-control", "max-age=0")
- if getFreshness(respHeaders, reqHeaders) != stale {
- t.Fatal("freshness isn't stale")
- }
- }
- func TestMinFreshWithExpires(t *testing.T) {
- resetTest()
- now := time.Now()
- respHeaders := http.Header{}
- respHeaders.Set("date", now.Format(time.RFC1123))
- respHeaders.Set("expires", now.Add(time.Duration(2)*time.Second).Format(time.RFC1123))
- reqHeaders := http.Header{}
- reqHeaders.Set("cache-control", "min-fresh=1")
- if getFreshness(respHeaders, reqHeaders) != fresh {
- t.Fatal("freshness isn't fresh")
- }
- reqHeaders = http.Header{}
- reqHeaders.Set("cache-control", "min-fresh=2")
- if getFreshness(respHeaders, reqHeaders) != stale {
- t.Fatal("freshness isn't stale")
- }
- }
- func TestEmptyMaxStale(t *testing.T) {
- resetTest()
- now := time.Now()
- respHeaders := http.Header{}
- respHeaders.Set("date", now.Format(time.RFC1123))
- respHeaders.Set("cache-control", "max-age=20")
- reqHeaders := http.Header{}
- reqHeaders.Set("cache-control", "max-stale")
- clock = &fakeClock{elapsed: 10 * time.Second}
- if getFreshness(respHeaders, reqHeaders) != fresh {
- t.Fatal("freshness isn't fresh")
- }
- clock = &fakeClock{elapsed: 60 * time.Second}
- if getFreshness(respHeaders, reqHeaders) != fresh {
- t.Fatal("freshness isn't fresh")
- }
- }
- func TestMaxStaleValue(t *testing.T) {
- resetTest()
- now := time.Now()
- respHeaders := http.Header{}
- respHeaders.Set("date", now.Format(time.RFC1123))
- respHeaders.Set("cache-control", "max-age=10")
- reqHeaders := http.Header{}
- reqHeaders.Set("cache-control", "max-stale=20")
- clock = &fakeClock{elapsed: 5 * time.Second}
- if getFreshness(respHeaders, reqHeaders) != fresh {
- t.Fatal("freshness isn't fresh")
- }
- clock = &fakeClock{elapsed: 15 * time.Second}
- if getFreshness(respHeaders, reqHeaders) != fresh {
- t.Fatal("freshness isn't fresh")
- }
- clock = &fakeClock{elapsed: 30 * time.Second}
- if getFreshness(respHeaders, reqHeaders) != stale {
- t.Fatal("freshness isn't stale")
- }
- }
- func containsHeader(headers []string, header string) bool {
- for _, v := range headers {
- if http.CanonicalHeaderKey(v) == http.CanonicalHeaderKey(header) {
- return true
- }
- }
- return false
- }
- func TestGetEndToEndHeaders(t *testing.T) {
- resetTest()
- var (
- headers http.Header
- end2end []string
- )
- headers = http.Header{}
- headers.Set("content-type", "text/html")
- headers.Set("te", "deflate")
- end2end = getEndToEndHeaders(headers)
- if !containsHeader(end2end, "content-type") {
- t.Fatal(`doesn't contain "content-type" header`)
- }
- if containsHeader(end2end, "te") {
- t.Fatal(`doesn't contain "te" header`)
- }
- headers = http.Header{}
- headers.Set("connection", "content-type")
- headers.Set("content-type", "text/csv")
- headers.Set("te", "deflate")
- end2end = getEndToEndHeaders(headers)
- if containsHeader(end2end, "connection") {
- t.Fatal(`doesn't contain "connection" header`)
- }
- if containsHeader(end2end, "content-type") {
- t.Fatal(`doesn't contain "content-type" header`)
- }
- if containsHeader(end2end, "te") {
- t.Fatal(`doesn't contain "te" header`)
- }
- headers = http.Header{}
- end2end = getEndToEndHeaders(headers)
- if len(end2end) != 0 {
- t.Fatal(`non-zero end2end headers`)
- }
- headers = http.Header{}
- headers.Set("connection", "content-type")
- end2end = getEndToEndHeaders(headers)
- if len(end2end) != 0 {
- t.Fatal(`non-zero end2end headers`)
- }
- }
- type transportMock struct {
- response *http.Response
- err error
- }
- func (t transportMock) RoundTrip(req *http.Request) (resp *http.Response, err error) {
- return t.response, t.err
- }
- func TestStaleIfErrorRequest(t *testing.T) {
- resetTest()
- now := time.Now()
- tmock := transportMock{
- response: &http.Response{
- Status: http.StatusText(http.StatusOK),
- StatusCode: http.StatusOK,
- Header: http.Header{
- "Date": []string{now.Format(time.RFC1123)},
- "Cache-Control": []string{"no-cache"},
- },
- Body: ioutil.NopCloser(bytes.NewBuffer([]byte("some data"))),
- },
- err: nil,
- }
- tp := NewMemoryCacheTransport()
- tp.Transport = &tmock
- // First time, response is cached on success
- r, _ := http.NewRequest("GET", "http://somewhere.com/", nil)
- r.Header.Set("Cache-Control", "stale-if-error")
- resp, err := tp.RoundTrip(r)
- if err != nil {
- t.Fatal(err)
- }
- if resp == nil {
- t.Fatal("resp is nil")
- }
- _, err = ioutil.ReadAll(resp.Body)
- if err != nil {
- t.Fatal(err)
- }
- // On failure, response is returned from the cache
- tmock.response = nil
- tmock.err = errors.New("some error")
- resp, err = tp.RoundTrip(r)
- if err != nil {
- t.Fatal(err)
- }
- if resp == nil {
- t.Fatal("resp is nil")
- }
- }
- func TestStaleIfErrorRequestLifetime(t *testing.T) {
- resetTest()
- now := time.Now()
- tmock := transportMock{
- response: &http.Response{
- Status: http.StatusText(http.StatusOK),
- StatusCode: http.StatusOK,
- Header: http.Header{
- "Date": []string{now.Format(time.RFC1123)},
- "Cache-Control": []string{"no-cache"},
- },
- Body: ioutil.NopCloser(bytes.NewBuffer([]byte("some data"))),
- },
- err: nil,
- }
- tp := NewMemoryCacheTransport()
- tp.Transport = &tmock
- // First time, response is cached on success
- r, _ := http.NewRequest("GET", "http://somewhere.com/", nil)
- r.Header.Set("Cache-Control", "stale-if-error=100")
- resp, err := tp.RoundTrip(r)
- if err != nil {
- t.Fatal(err)
- }
- if resp == nil {
- t.Fatal("resp is nil")
- }
- _, err = ioutil.ReadAll(resp.Body)
- if err != nil {
- t.Fatal(err)
- }
- // On failure, response is returned from the cache
- tmock.response = nil
- tmock.err = errors.New("some error")
- resp, err = tp.RoundTrip(r)
- if err != nil {
- t.Fatal(err)
- }
- if resp == nil {
- t.Fatal("resp is nil")
- }
- // Same for http errors
- tmock.response = &http.Response{StatusCode: http.StatusInternalServerError}
- tmock.err = nil
- resp, err = tp.RoundTrip(r)
- if err != nil {
- t.Fatal(err)
- }
- if resp == nil {
- t.Fatal("resp is nil")
- }
- // If failure last more than max stale, error is returned
- clock = &fakeClock{elapsed: 200 * time.Second}
- _, err = tp.RoundTrip(r)
- if err != tmock.err {
- t.Fatalf("got err %v, want %v", err, tmock.err)
- }
- }
- func TestStaleIfErrorResponse(t *testing.T) {
- resetTest()
- now := time.Now()
- tmock := transportMock{
- response: &http.Response{
- Status: http.StatusText(http.StatusOK),
- StatusCode: http.StatusOK,
- Header: http.Header{
- "Date": []string{now.Format(time.RFC1123)},
- "Cache-Control": []string{"no-cache, stale-if-error"},
- },
- Body: ioutil.NopCloser(bytes.NewBuffer([]byte("some data"))),
- },
- err: nil,
- }
- tp := NewMemoryCacheTransport()
- tp.Transport = &tmock
- // First time, response is cached on success
- r, _ := http.NewRequest("GET", "http://somewhere.com/", nil)
- resp, err := tp.RoundTrip(r)
- if err != nil {
- t.Fatal(err)
- }
- if resp == nil {
- t.Fatal("resp is nil")
- }
- _, err = ioutil.ReadAll(resp.Body)
- if err != nil {
- t.Fatal(err)
- }
- // On failure, response is returned from the cache
- tmock.response = nil
- tmock.err = errors.New("some error")
- resp, err = tp.RoundTrip(r)
- if err != nil {
- t.Fatal(err)
- }
- if resp == nil {
- t.Fatal("resp is nil")
- }
- }
- func TestStaleIfErrorResponseLifetime(t *testing.T) {
- resetTest()
- now := time.Now()
- tmock := transportMock{
- response: &http.Response{
- Status: http.StatusText(http.StatusOK),
- StatusCode: http.StatusOK,
- Header: http.Header{
- "Date": []string{now.Format(time.RFC1123)},
- "Cache-Control": []string{"no-cache, stale-if-error=100"},
- },
- Body: ioutil.NopCloser(bytes.NewBuffer([]byte("some data"))),
- },
- err: nil,
- }
- tp := NewMemoryCacheTransport()
- tp.Transport = &tmock
- // First time, response is cached on success
- r, _ := http.NewRequest("GET", "http://somewhere.com/", nil)
- resp, err := tp.RoundTrip(r)
- if err != nil {
- t.Fatal(err)
- }
- if resp == nil {
- t.Fatal("resp is nil")
- }
- _, err = ioutil.ReadAll(resp.Body)
- if err != nil {
- t.Fatal(err)
- }
- // On failure, response is returned from the cache
- tmock.response = nil
- tmock.err = errors.New("some error")
- resp, err = tp.RoundTrip(r)
- if err != nil {
- t.Fatal(err)
- }
- if resp == nil {
- t.Fatal("resp is nil")
- }
- // If failure last more than max stale, error is returned
- clock = &fakeClock{elapsed: 200 * time.Second}
- _, err = tp.RoundTrip(r)
- if err != tmock.err {
- t.Fatalf("got err %v, want %v", err, tmock.err)
- }
- }
- // Test that http.Client.Timeout is respected when cache transport is used.
- // That is so as long as request cancellation is propagated correctly.
- // In the past, that required CancelRequest to be implemented correctly,
- // but modern http.Client uses Request.Cancel (or request context) instead,
- // so we don't have to do anything.
- func TestClientTimeout(t *testing.T) {
- if testing.Short() {
- t.Skip("skipping timeout test in short mode") // Because it takes at least 3 seconds to run.
- }
- resetTest()
- client := &http.Client{
- Transport: NewMemoryCacheTransport(),
- Timeout: time.Second,
- }
- started := time.Now()
- resp, err := client.Get(s.server.URL + "/3seconds")
- taken := time.Since(started)
- if err == nil {
- t.Error("got nil error, want timeout error")
- }
- if resp != nil {
- t.Error("got non-nil resp, want nil resp")
- }
- if taken >= 2*time.Second {
- t.Error("client.Do took 2+ seconds, want < 2 seconds")
- }
- }
|