Merge pull request #346 from bdwyertech/patch-1

Oversize Cookie Alert
bitly · Mar 29, 2017 · 107b481 · 107b481
2 parents cd0d13e + 3379e05
commit 107b481
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/oauthproxy.go b/oauthproxy.go
@@ -257,6 +257,10 @@ func (p *OAuthProxy) MakeCookie(req *http.Request, value string, expiration time
 
 	if value != "" {
 		value = cookie.SignedValue(p.CookieSeed, p.CookieName, value, now)
+		if len(value) > 4096 {
+			// Cookies cannot be larger than 4kb
+			log.Printf("WARNING - Cookie Size: %d bytes", len(value))
+		}
 	}
 	return &http.Cookie{
 		Name:     p.CookieName,