You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by no...@apache.org on 2019/01/05 08:52:17 UTC

[2/2] lucene-solr:branch_7x: SOLR-12514: FIxed the test and another bug

SOLR-12514: FIxed the test and another bug


Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/ecd4a3b4
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/ecd4a3b4
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/ecd4a3b4

Branch: refs/heads/branch_7x
Commit: ecd4a3b49c0eb51b688997654f345efe0d92c85c
Parents: e085fcd
Author: Noble Paul <no...@apache.org>
Authored: Sat Jan 5 19:52:02 2019 +1100
Committer: Noble Paul <no...@apache.org>
Committed: Sat Jan 5 19:52:02 2019 +1100

----------------------------------------------------------------------
 .../org/apache/solr/security/HttpParamDelegationTokenPlugin.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/ecd4a3b4/solr/core/src/test/org/apache/solr/security/HttpParamDelegationTokenPlugin.java
----------------------------------------------------------------------
diff --git a/solr/core/src/test/org/apache/solr/security/HttpParamDelegationTokenPlugin.java b/solr/core/src/test/org/apache/solr/security/HttpParamDelegationTokenPlugin.java
index 7a4f69f..4ea07b4 100644
--- a/solr/core/src/test/org/apache/solr/security/HttpParamDelegationTokenPlugin.java
+++ b/solr/core/src/test/org/apache/solr/security/HttpParamDelegationTokenPlugin.java
@@ -76,7 +76,7 @@ public class HttpParamDelegationTokenPlugin extends KerberosPlugin {
       SolrRequestInfo reqInfo = SolrRequestInfo.getRequestInfo();
       String usr;
       if (reqInfo != null) {
-        Principal principal = reqInfo.getReq().getUserPrincipal();
+        Principal principal = reqInfo.getUserPrincipal();
         if (principal == null) {
           //this had a request but not authenticated
           //so we don't not need to set a principal