tree 6de38346a795d151127aba1c80df0dafce1e8464
parent 56fdb27532ffe5f8f0d2c7c8b1f0b4d1da481f10
author Tuo Shan <sturbo89@gmail.com> 1461783440 -0400
committer Andrew Gerrand <adg@golang.org> 1461901064 +0000

gosrc: include package updated time in the request header.

gddo-server: pass pdoc instead of nil in the refresh call.

The package updated time is included in the header to check if it is
recently updated when requesting from VCS. Currently we use http ETag
as a general way to compare changes, which still consumes one API
request rate when requesting from Github. However, Github accepts
conditional requests using "If-Modified-Since" to check recent updates
time without counting against the rate limit if a 304 is received.

Package document is included in the refresh call to avoid unlimited
manual refreshing. The current design to allow such refresh to force
updating the package seems unnecessary.

These changes are targeting at reducing Github API request rate in order
to give us more head room when we do batch updates to do type analysis.

Change-Id: I4ec81c92662cd23dc96e5a5e141f1945b127db2c
Reviewed-on: https://go-review.googlesource.com/22506
Reviewed-by: Alan Donovan <adonovan@google.com>
Reviewed-by: Andrew Gerrand <adg@golang.org>