Merge pull request #5791 from dstufft/zero-cache-time-simple

Only revalidate /simple/ pages instead of caching for 10 minutes

news/5670.bugfix

@@ -0,0 +1,2 @@
+Always revalidate cached simple API pages instead of blindly caching them for up to 10
+minutes.

src/pip/_internal/index.py

@@ -783,7 +783,20 @@ def get_page(cls, link, skip_archives=True, session=None):
                 url,
                 headers={
                     "Accept": "text/html",
-                    "Cache-Control": "max-age=600",
+                    # We don't want to blindly returned cached data for
+                    # /simple/, because authors generally expecting that
+                    # twine upload && pip install will function, but if
+                    # they've done a pip install in the last ~10 minutes
+                    # it won't. Thus by setting this to zero we will not
+                    # blindly use any cached data, however the benefit of
+                    # using max-age=0 instead of no-cache, is that we will
+                    # still support conditional requests, so we will still
+                    # minimize traffic sent in cases where the page hasn't
+                    # changed at all, we will just always incur the round
+                    # trip for the conditional GET now instead of only
+                    # once per 10 minutes.
+                    # For more information, please see pypa/pip#5670.
+                    "Cache-Control": "max-age=0",
                 },
             )
             resp.raise_for_status()