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 2015/09/02 17:15:59 UTC

svn commit: r1700841 - in /lucene/dev/branches/branch_5x: ./ solr/ solr/CHANGES.txt solr/core/ solr/core/src/java/org/apache/solr/core/RequestHandlers.java

Author: noble
Date: Wed Sep  2 15:15:59 2015
New Revision: 1700841

URL: http://svn.apache.org/r1700841
Log:
SOLR-7984: wrong and misleading error message 'no default request handler is registered'

Modified:
    lucene/dev/branches/branch_5x/   (props changed)
    lucene/dev/branches/branch_5x/solr/   (props changed)
    lucene/dev/branches/branch_5x/solr/CHANGES.txt   (contents, props changed)
    lucene/dev/branches/branch_5x/solr/core/   (props changed)
    lucene/dev/branches/branch_5x/solr/core/src/java/org/apache/solr/core/RequestHandlers.java

Modified: lucene/dev/branches/branch_5x/solr/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_5x/solr/CHANGES.txt?rev=1700841&r1=1700840&r2=1700841&view=diff
==============================================================================
--- lucene/dev/branches/branch_5x/solr/CHANGES.txt (original)
+++ lucene/dev/branches/branch_5x/solr/CHANGES.txt Wed Sep  2 15:15:59 2015
@@ -99,6 +99,8 @@ Bug Fixes
 
 * SOLR-8004: RuleBasedAuthorization plugin does not work for the collection-admin-edit permission (noble)
 
+* SOLR-7984: wrong and misleading error message 'no default request handler is registered' (noble, hossman)
+
 
 Optimizations
 ----------------------

Modified: lucene/dev/branches/branch_5x/solr/core/src/java/org/apache/solr/core/RequestHandlers.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_5x/solr/core/src/java/org/apache/solr/core/RequestHandlers.java?rev=1700841&r1=1700840&r2=1700841&view=diff
==============================================================================
--- lucene/dev/branches/branch_5x/solr/core/src/java/org/apache/solr/core/RequestHandlers.java (original)
+++ lucene/dev/branches/branch_5x/solr/core/src/java/org/apache/solr/core/RequestHandlers.java Wed Sep  2 15:15:59 2015
@@ -130,12 +130,11 @@ public final class RequestHandlers {
     handlers.init(Collections.EMPTY_MAP,core, modifiedInfos);
     handlers.alias(handlers.getDefault(), "");
     log.info("Registered paths: {}" , StrUtils.join(new ArrayList<>(handlers.keySet()) , ',' ));
-    if(!handlers.alias( "/select","")){
-      if(!handlers.alias( "standard","")){
+    if (handlers.get("") == null && !handlers.alias("/select", "")) {
+      if (handlers.get("") == null && !handlers.alias("standard", "")) {
         log.warn("no default request handler is registered (either '/select' or 'standard')");
       }
     }
-
   }
 
   private PluginInfo applyInitParams(SolrConfig config, PluginInfo info) {