summaryrefslogtreecommitdiff
path: root/CONTRIBUTING.md
diff options
context:
space:
mode:
authorRémy Coutable <remy@rymai.me>2016-10-07 10:35:03 +0000
committerRémy Coutable <remy@rymai.me>2016-10-07 10:35:03 +0000
commit0876b46024be019d9ef9846045d3941dcdf90981 (patch)
tree84ea7e0cd4a377963a4e4af28f683bb1fa323912 /CONTRIBUTING.md
parent1be151621721d9e89c4c6514b8523e999e5e0fed (diff)
parent1c462cf7d6d61aebac9b909102f0e794cc9e409a (diff)
downloadgitlab-ce-0876b46024be019d9ef9846045d3941dcdf90981.tar.gz
Merge branch 'memoize_shell_secret_token' into 'master'
Memoize Github::Shell's secret token ## What does this MR do? `API::Helpers#secret_token` was reading the secret file on every invocation. This MR reads the file in the `gitlab_shell_secret_token.rb` initializer and saves it as a class variable at `Gitlab::Shell.secret_token` ## Are there points in the code the reviewer needs to double check? - I'm not sure if the use of `cattr_accessor` is the best approach, or if should be moved into the `class << self` block? - Should `API::Helpers#secret_token` be removed in favor of using `Gitlab::Shell.secret_token`? ## Why was this MR needed? Performance optimization. Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/22510 See merge request !6599
Diffstat (limited to 'CONTRIBUTING.md')
0 files changed, 0 insertions, 0 deletions