diff options
| author | Yorick Peterse <yorickpeterse@gmail.com> | 2018-06-19 19:03:25 +0200 |
|---|---|---|
| committer | Yorick Peterse <yorickpeterse@gmail.com> | 2018-06-21 16:56:40 +0200 |
| commit | 9e29408ee6072f2559899582b3300a6de54069ff (patch) | |
| tree | 921e3aa0bf3ebe9b61b74973c239b8108de4175b /lib/api | |
| parent | d229e7d3f50fd1f36bacecd55bb74d0b1ab44cbe (diff) | |
| download | gitlab-ce-9e29408ee6072f2559899582b3300a6de54069ff.tar.gz | |
Don't expose project names in various counters
Various counters would expose either project names, or full project
paths (e.g. "gitlab-org/gitlab-ce"). This commit changes various places
where we use "add_event" so we no longer expose (potentially) private
information.
Diffstat (limited to 'lib/api')
| -rw-r--r-- | lib/api/runner.rb | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/lib/api/runner.rb b/lib/api/runner.rb index 96a02914faa..80b37dce118 100644 --- a/lib/api/runner.rb +++ b/lib/api/runner.rb @@ -101,8 +101,7 @@ module API if result.valid? if result.build - Gitlab::Metrics.add_event(:build_found, - project: result.build.project.full_path) + Gitlab::Metrics.add_event(:build_found) present result.build, with: Entities::JobRequest::Response else Gitlab::Metrics.add_event(:build_not_found) @@ -133,8 +132,7 @@ module API job.trace.set(params[:trace]) if params[:trace] - Gitlab::Metrics.add_event(:update_build, - project: job.project.full_path) + Gitlab::Metrics.add_event(:update_build) case params[:state].to_s when 'running' |