Fix missing increment on recaptcha counter
This commit is contained in:
		
							parent
							
								
									a4a666bb80
								
							
						
					
					
						commit
						61b3cecd17
					
				| 
						 | 
					@ -132,7 +132,8 @@ public class RecaptchaClient {
 | 
				
			||||||
        Metrics.counter(ASSESSMENT_REASON_COUNTER_NAME,
 | 
					        Metrics.counter(ASSESSMENT_REASON_COUNTER_NAME,
 | 
				
			||||||
            "action", String.valueOf(expectedAction),
 | 
					            "action", String.valueOf(expectedAction),
 | 
				
			||||||
            "score", scoreString(assessment.getRiskAnalysis().getScore()),
 | 
					            "score", scoreString(assessment.getRiskAnalysis().getScore()),
 | 
				
			||||||
            "reason", reason.name());
 | 
					            "reason", reason.name())
 | 
				
			||||||
 | 
					            .increment();
 | 
				
			||||||
      }
 | 
					      }
 | 
				
			||||||
      final float score = assessment.getRiskAnalysis().getScore();
 | 
					      final float score = assessment.getRiskAnalysis().getScore();
 | 
				
			||||||
      return new AssessmentResult(
 | 
					      return new AssessmentResult(
 | 
				
			||||||
| 
						 | 
					@ -142,7 +143,8 @@ public class RecaptchaClient {
 | 
				
			||||||
    } else {
 | 
					    } else {
 | 
				
			||||||
      Metrics.counter(INVALID_REASON_COUNTER_NAME,
 | 
					      Metrics.counter(INVALID_REASON_COUNTER_NAME,
 | 
				
			||||||
          "action", String.valueOf(expectedAction),
 | 
					          "action", String.valueOf(expectedAction),
 | 
				
			||||||
          "reason", assessment.getTokenProperties().getInvalidReason().name());
 | 
					          "reason", assessment.getTokenProperties().getInvalidReason().name())
 | 
				
			||||||
 | 
					          .increment();
 | 
				
			||||||
      return AssessmentResult.invalid();
 | 
					      return AssessmentResult.invalid();
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
  }
 | 
					  }
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in New Issue