This is an automated email from the ASF dual-hosted git repository.
dsmiley pushed a commit to branch branch_10x
in repository https://gitbox.apache.org/repos/asf/solr.git
The following commit(s) were added to refs/heads/branch_10x by this push:
new 80426599296 minor: AuthorizationFailure is now a record (#4198)
80426599296 is described below
commit 8042659929689bb6762b44132a51b2af62ad1ad0
Author: David Smiley <[email protected]>
AuthorDate: Wed Mar 11 08:26:18 2026 -0400
minor: AuthorizationFailure is now a record (#4198)
---
.../org/apache/solr/jersey/SolrRequestAuthorizer.java | 2 +-
.../org/apache/solr/security/AuthorizationUtils.java | 18 +-----------------
.../src/java/org/apache/solr/servlet/HttpSolrCall.java | 2 +-
3 files changed, 3 insertions(+), 19 deletions(-)
diff --git
a/solr/core/src/java/org/apache/solr/jersey/SolrRequestAuthorizer.java
b/solr/core/src/java/org/apache/solr/jersey/SolrRequestAuthorizer.java
index 7fa7a05f2a9..1a96f534310 100644
--- a/solr/core/src/java/org/apache/solr/jersey/SolrRequestAuthorizer.java
+++ b/solr/core/src/java/org/apache/solr/jersey/SolrRequestAuthorizer.java
@@ -89,7 +89,7 @@ public class SolrRequestAuthorizer implements
ContainerRequestFilter {
AuthorizationUtils.authorize(servletRequest, servletResponse,
coreContainer, authzContext);
if (authzFailure != null) {
final Response failureResponse =
-
Response.status(authzFailure.getStatusCode()).entity(authzFailure.getMessage()).build();
+
Response.status(authzFailure.statusCode()).entity(authzFailure.message()).build();
requestContext.abortWith(failureResponse);
}
}
diff --git
a/solr/core/src/java/org/apache/solr/security/AuthorizationUtils.java
b/solr/core/src/java/org/apache/solr/security/AuthorizationUtils.java
index d551c4eddc8..a316dbdf65d 100644
--- a/solr/core/src/java/org/apache/solr/security/AuthorizationUtils.java
+++ b/solr/core/src/java/org/apache/solr/security/AuthorizationUtils.java
@@ -43,23 +43,7 @@ public class AuthorizationUtils {
/* Private ctor prevents instantiation */
}
- public static class AuthorizationFailure {
- private final int statusCode;
- private final String message;
-
- public AuthorizationFailure(int statusCode, String message) {
- this.statusCode = statusCode;
- this.message = message;
- }
-
- public int getStatusCode() {
- return statusCode;
- }
-
- public String getMessage() {
- return message;
- }
- }
+ public record AuthorizationFailure(int statusCode, String message) {}
public static AuthorizationFailure authorize(
HttpServletRequest servletReq,
diff --git a/solr/core/src/java/org/apache/solr/servlet/HttpSolrCall.java
b/solr/core/src/java/org/apache/solr/servlet/HttpSolrCall.java
index c289ca6a3a9..5e2fc97d163 100644
--- a/solr/core/src/java/org/apache/solr/servlet/HttpSolrCall.java
+++ b/solr/core/src/java/org/apache/solr/servlet/HttpSolrCall.java
@@ -450,7 +450,7 @@ public class HttpSolrCall {
AuthorizationUtils.AuthorizationFailure authzFailure =
AuthorizationUtils.authorize(req, response, cores, authzContext);
if (authzFailure != null) {
- sendError(authzFailure.getStatusCode(), authzFailure.getMessage());
+ sendError(authzFailure.statusCode(), authzFailure.message());
return RETURN;
}
}