back round 0 view raw

appview: cache: bump session cache TTL to 7 days #223

merged

opened by

oppi.li 6 months ago targeting master from push-upnknovvnoqr

Signed-off-by: oppiliappan me@oppi.li

options

unified split

Changed files

+3 -4

appview

cache

session

store.go

middleware

middleware.go

+2 -3

appview/cache/session/store.go

···

       54
        
       		return err

     

       55
        
       	}

     

       56
        
       

     

       57
       -
       	// set with ttl (expires in + buffer)

     

       58
       -
       	expiry, _ := time.Parse(time.RFC3339, session.Expiry)

     

       59
       -
       	ttl := time.Until(expiry) + time.Minute

     

       60
        
       

     

       61
        
       	return s.cache.Set(ctx, key, data, ttl).Err()

     

       62
        
       }

···

       54
        
       		return err

     

       55
        
       	}

     

       56
        
       

     

       57
       +
       	// set with ttl (7 days)

     

       58
       +
       	ttl := 7 * 24 * time.Hour

     

       0
        
       
     

       59
        
       

     

       60
        
       	return s.cache.Set(ctx, key, data, ttl).Err()

     

       61
        
       }

+1 -1

appview/middleware/middleware.go

···

       58
        
       

     

       59
        
       			_, auth, err := a.GetSession(r)

     

       60
        
       			if err != nil {

     

       61
       -
       				log.Printf("not logged in, redirecting")

     

       62
        
       				redirectFunc(w, r)

     

       63
        
       				return

     

       64
        
       			}

···

       58
        
       

     

       59
        
       			_, auth, err := a.GetSession(r)

     

       60
        
       			if err != nil {

     

       61
       +
       				log.Println("not logged in, redirecting", "err", err)

     

       62
        
       				redirectFunc(w, r)

     

       63
        
       				return

     

       64
        
       			}