Skip to content

Commit 676fb14

Browse files
committed
fix: replace joda-time with java.time in ThrottlingOpenApiViolationHandler
swagger-request-validator 3.0.0 no longer brings joda-time in transitively.
1 parent 30b726c commit 676fb14

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

‎openapi-validation-core/src/main/java/com/getyourguide/openapi/validation/core/log/ThrottlingOpenApiViolationHandler.java‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,17 +2,17 @@
22

33
import com.getyourguide.openapi.validation.api.log.OpenApiViolationHandler;
44
import com.getyourguide.openapi.validation.api.model.OpenApiViolation;
5+
import java.time.Instant;
56
import java.util.Map;
67
import java.util.concurrent.ConcurrentHashMap;
78
import lombok.AllArgsConstructor;
89
import lombok.NonNull;
9-
import org.joda.time.DateTime;
1010

1111
@AllArgsConstructor
1212
public class ThrottlingOpenApiViolationHandler implements OpenApiViolationHandler {
1313
private final OpenApiViolationHandler delegate;
1414
private final int waitSeconds;
15-
private final Map<String, DateTime> loggedMessages = new ConcurrentHashMap<>();
15+
private final Map<String, Instant> loggedMessages = new ConcurrentHashMap<>();
1616

1717
@Override
1818
public void onOpenApiViolation(OpenApiViolation violation) {
@@ -25,13 +25,13 @@ public void onOpenApiViolation(OpenApiViolation violation) {
2525
}
2626

2727
private void registerLoggedMessage(OpenApiViolation openApiViolation) {
28-
loggedMessages.put(buildKey(openApiViolation), DateTime.now());
28+
loggedMessages.put(buildKey(openApiViolation), Instant.now());
2929
}
3030

3131
private boolean isThrottled(OpenApiViolation openApiViolation) {
3232
var key = buildKey(openApiViolation);
3333
var lastLoggedTime = loggedMessages.get(key);
34-
return lastLoggedTime != null && lastLoggedTime.plusSeconds(waitSeconds).isAfterNow();
34+
return lastLoggedTime != null && lastLoggedTime.plusSeconds(waitSeconds).isAfter(Instant.now());
3535
}
3636

3737
@NonNull

0 commit comments

Comments
 (0)