Skip to content

Commit

Permalink
Merge pull request #57 from craigatk/remove-metric
Browse files Browse the repository at this point in the history
(chore) Removing unneeded metric
  • Loading branch information
craigatk authored Apr 9, 2020
2 parents 52dd1cf + e958070 commit f318d78
Showing 1 changed file with 0 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,6 @@ fun Route.results(
authService: AuthService,
metricRegistry: MeterRegistry
) {
val groupedResultsSuccessCounter = metricRegistry.counter("routes_grouped_results_success")

post("/results") {
if (!authService.isAuthValid(call.request.header(AuthConfig.PublishToken))) {
call.respond(HttpStatusCode.Unauthorized)
Expand All @@ -62,7 +60,6 @@ fun Route.results(

if (groupedResultsBlob.isNotBlank()) {
val publicId = groupedTestResultsService.persistTestResultsAsync(groupedResultsBlob)
groupedResultsSuccessCounter.increment()

call.respond(HttpStatusCode.OK, SaveResultsResponse(publicId.id, "/tests/${publicId.id}"))
} else {
Expand Down

0 comments on commit f318d78

Please sign in to comment.