Enable HTTP caching by default, and disable it conditionally.

Review Request #13040 — Created May 11, 2023 and submitted — Latest diff uploaded

Information

RBTools
release-5.x

Reviewers

RBTools's HTTP caching is conditional based on the version of Review
Board. In versions prior to 2.0.14, caching was broken in various ways,
so RBTools only enables it if the root resource claims a newer version.
By default, though, caching was disabled, meaning the root resource
never went through the caching support.

Since most companies are using versions far newer than the minimum of
2.0.14, this change flips the logic around. Caching is now enabled by
default for all resources, and is then disabled if the root resource
claims an earlier version. This gives us the best performance available
for most users.

Some API transport and response testing infrastructure had to be updated
to test the new code paths. Those are included in this change.

Unit tests pass.

Tested posting changes against reviews.reviewboard.org and local
servers.

Commits

Files

    Loading...