Minor: Return exception instead of throwing it in .orElseThrow()
Gets rid of an IntelliJ warning about it. No difference in behavior, there is a test that already covers this path.
This commit is contained in:
parent
ed72d7f9ec
commit
2d314e5309
|
@ -87,7 +87,7 @@ public class CaptchaChecker {
|
||||||
final Action parsedAction = Action.parse(action)
|
final Action parsedAction = Action.parse(action)
|
||||||
.orElseThrow(() -> {
|
.orElseThrow(() -> {
|
||||||
Metrics.counter(INVALID_ACTION_COUNTER_NAME, "action", action).increment();
|
Metrics.counter(INVALID_ACTION_COUNTER_NAME, "action", action).increment();
|
||||||
throw new BadRequestException("invalid captcha action");
|
return new BadRequestException("invalid captcha action");
|
||||||
});
|
});
|
||||||
|
|
||||||
if (!parsedAction.equals(expectedAction)) {
|
if (!parsedAction.equals(expectedAction)) {
|
||||||
|
|
Loading…
Reference in New Issue