diff options
author | Douwe Maan <douwe@selenight.nl> | 2018-02-16 15:15:32 +0100 |
---|---|---|
committer | Stan Hu <stanhu@gmail.com> | 2018-02-16 22:28:08 -0800 |
commit | 8b727fea81b19f12366894c4a4b981250c549833 (patch) | |
tree | 89ed1215f3f80907a33ed2479ceefad788e91470 | |
parent | b236348388c46c0550ec6844df35ec2689c4060b (diff) | |
download | gitlab-ce-8b727fea81b19f12366894c4a4b981250c549833.tar.gz |
Don't cache a nil repository root ref to prevent caching issuesdm-dont-cache-nil-root-ref
-rw-r--r-- | app/models/repository.rb | 8 | ||||
-rw-r--r-- | changelogs/unreleased/dm-dont-cache-nil-root-ref.yml | 5 |
2 files changed, 7 insertions, 6 deletions
diff --git a/app/models/repository.rb b/app/models/repository.rb index 4f754b11da4..299a3f32a85 100644 --- a/app/models/repository.rb +++ b/app/models/repository.rb @@ -492,12 +492,8 @@ class Repository end def root_ref - if raw_repository - raw_repository.root_ref - else - # When the repo does not exist we raise this error so no data is cached. - raise Gitlab::Git::Repository::NoRepository - end + # When the repo does not exist, or there is no root ref, we raise this error so no data is cached. + raw_repository&.root_ref or raise Gitlab::Git::Repository::NoRepository # rubocop:disable Style/AndOr end cache_method :root_ref diff --git a/changelogs/unreleased/dm-dont-cache-nil-root-ref.yml b/changelogs/unreleased/dm-dont-cache-nil-root-ref.yml new file mode 100644 index 00000000000..4dab7d0ffca --- /dev/null +++ b/changelogs/unreleased/dm-dont-cache-nil-root-ref.yml @@ -0,0 +1,5 @@ +--- +title: Don't cache a nil repository root ref to prevent caching issues +merge_request: +author: +type: fixed |