Skip to content

Add latest commit hash to compare cache key

Robert May requested to merge compare-caching-fix into master

What does this MR do and why?

Describe in detail what your merge request does and why.

This should hopefully resolve an issue where adding a new commit via the API doesn't expire the rate limit cache on the repository compare endpoint.

Testing this locally is really quite awkward, and I've hopefully covered it with the spec, but it is quite inconsistent to reproduce somehow.

Related #339132 (closed)

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports

Loading