commit 571e31d04401dbcd9f0a6fbd479148a49245e255
author QUICHE team <quiche-dev@google.com> Thu Oct 29 08:18:41 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Thu Oct 29 08:29:31 2020 -0700
tree 95c9c471bffb5f5b9fe5d4a8881ad9ec2f2f5717
parent c63780bad4e9e1ea57baaa5574ec4186b81b5bef

Update s-maxage separately from max-age for CACHE_ALL_STATIC client_ttl

As part of the change, we refactor gfe::header_utils::GetMaxAge() into another
GetClientMaxAge() version that doesn't look at s_maxage, so we can use that
to properly clamp just the client portion with client_ttl_secs.

Also, fix an odd little glitch that max-age=-1 (and s-maxage=-1) was
correctly identified by the code as illegal, but not max-age=-2 or smaller.

Protected by gfe2_reloadable_flag_clamp_smaxage.

PiperOrigin-RevId: 339670897
Change-Id: Ice213d4ba503d92fb077ebd25ee0dc84136c0dda