You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ja...@apache.org on 2021/11/18 11:58:34 UTC
[lucene-solr] branch branch_8_11 updated: SOLR-15804 Allow content-types with ; charset in ShowFileRequestHandler
This is an automated email from the ASF dual-hosted git repository.
janhoy pushed a commit to branch branch_8_11
in repository https://gitbox.apache.org/repos/asf/lucene-solr.git
The following commit(s) were added to refs/heads/branch_8_11 by this push:
new 04bf333 SOLR-15804 Allow content-types with ;charset in ShowFileRequestHandler
04bf333 is described below
commit 04bf333b338cb51530bd3b828d15c5fdcfe4814f
Author: Jan Høydahl <ja...@apache.org>
AuthorDate: Thu Nov 18 12:49:53 2021 +0100
SOLR-15804 Allow content-types with ;charset in ShowFileRequestHandler
* Request .html as text/xml, .css as text/plain and .js as application/javascript
* Highlight css as txt (crashed browser)
* Set extension to 'xml' for managed-schema instead of 'text/xml'
* Clean up 8.11.1 changes section, removing 'Build' and 'Other changes'
Cherry picked from branch_8x 4e5b7fd81d2add1acf0ec15539cc2d0453d754dd
---
solr/CHANGES.txt | 4 +++-
.../org/apache/solr/handler/admin/ShowFileRequestHandler.java | 10 ++++++++--
.../apache/solr/handler/admin/ShowFileRequestHandlerTest.java | 7 +++++++
solr/webapp/web/js/angular/controllers/files.js | 6 +++---
4 files changed, 21 insertions(+), 6 deletions(-)
diff --git a/solr/CHANGES.txt b/solr/CHANGES.txt
index 97af064..7332de5 100644
--- a/solr/CHANGES.txt
+++ b/solr/CHANGES.txt
@@ -12,9 +12,11 @@ Bug Fixes
---------------------
* SOLR-15635: Don't close hooks twice when SolrRequestInfo is cleared twice; or /export with classic join
closed fromCore if provided (Mikhail Khludnev, David Smiley)
-
+
* SOLR-15795: Fix REPLACENODE to not use source node when choosing a target node for new replicas (Houston Putman)
+* SOLR-15804: Admin UI once again can show files in the Core/Collection -> Files screen. Fixed regression from 8.11.0 (Karl Stoney, janhoy)
+
================== 8.11.0 ==================
Consult the LUCENE_CHANGES.txt file for additional, low level, changes in this release.
diff --git a/solr/core/src/java/org/apache/solr/handler/admin/ShowFileRequestHandler.java b/solr/core/src/java/org/apache/solr/handler/admin/ShowFileRequestHandler.java
index c3f40e8..3654b93 100644
--- a/solr/core/src/java/org/apache/solr/handler/admin/ShowFileRequestHandler.java
+++ b/solr/core/src/java/org/apache/solr/handler/admin/ShowFileRequestHandler.java
@@ -96,11 +96,16 @@ public class ShowFileRequestHandler extends RequestHandlerBase implements Permis
{
public static final String HIDDEN = "hidden";
public static final String USE_CONTENT_TYPE = "contentType";
+ private static final Set<String> KNOWN_MIME_TYPES;
protected Set<String> hiddenFiles;
private static final Logger log = LoggerFactory.getLogger(MethodHandles.lookup().lookupClass());
+ static {
+ KNOWN_MIME_TYPES = new HashSet<>(MimeTypes.getKnownMimeTypes());
+ KNOWN_MIME_TYPES.add("text/xml");
+ }
public ShowFileRequestHandler()
{
@@ -265,10 +270,11 @@ public class ShowFileRequestHandler extends RequestHandlerBase implements Permis
log.debug("No contentType specified");
return null;
}
- if (!MimeTypes.getKnownMimeTypes().contains(contentType)) {
+ String rawContentType = contentType.split(";")[0].trim().toLowerCase(Locale.ROOT); // Strip away charset part
+ if (!KNOWN_MIME_TYPES.contains(rawContentType)) {
throw new SolrException(ErrorCode.BAD_REQUEST, "Requested content type '" + contentType + "' is not supported.");
}
- if (contentType.toLowerCase(Locale.ROOT).contains("html")) {
+ if (rawContentType.contains("html")) {
log.info("Using text/plain instead of {}", contentType);
return "text/plain";
}
diff --git a/solr/core/src/test/org/apache/solr/handler/admin/ShowFileRequestHandlerTest.java b/solr/core/src/test/org/apache/solr/handler/admin/ShowFileRequestHandlerTest.java
index 0e78329..ccdb465 100644
--- a/solr/core/src/test/org/apache/solr/handler/admin/ShowFileRequestHandlerTest.java
+++ b/solr/core/src/test/org/apache/solr/handler/admin/ShowFileRequestHandlerTest.java
@@ -171,6 +171,13 @@ public class ShowFileRequestHandlerTest extends SolrJettyTestBase {
assertEquals("text/plain", ShowFileRequestHandler.getSafeContentType("text/html"));
assertEquals("text/plain", ShowFileRequestHandler.getSafeContentType("application/xhtml+xml"));
+ // Content-type with charset
+ assertEquals("text/csv ; charset=utf-8", ShowFileRequestHandler.getSafeContentType("text/csv ; charset=utf-8"));
+ assertEquals("text/xml;charset=utf-8", ShowFileRequestHandler.getSafeContentType("text/xml;charset=utf-8"));
+
+ // Null
+ assertNull(ShowFileRequestHandler.getSafeContentType(null));
+
// Non-known content types are rejected with 400 error
expectThrows(SolrException.class, () -> ShowFileRequestHandler.getSafeContentType("foo/bar"));
}
diff --git a/solr/webapp/web/js/angular/controllers/files.js b/solr/webapp/web/js/angular/controllers/files.js
index 76f7dd3..9275466 100644
--- a/solr/webapp/web/js/angular/controllers/files.js
+++ b/solr/webapp/web/js/angular/controllers/files.js
@@ -15,8 +15,8 @@
limitations under the License.
*/
-var contentTypeMap = { xml : 'text/xml', html : 'text/html', js : 'text/javascript', json : 'application/json', 'css' : 'text/css' };
-var languages = {js: "javascript", xml:"xml", xsl:"xml", vm: "xml", html: "xml", json: "json", css: "css"};
+var contentTypeMap = { xml : 'application/xml', html : 'application/xml', js : 'application/javascript', json : 'application/json', css : 'text/plain' };
+var languages = {js: "javascript", xml:"xml", xsl:"xml", vm: "xml", html: "xml", json: "json"};
solrAdminApp.controller('FilesController',
function($scope, $rootScope, $routeParams, $location, Files, Constants) {
@@ -71,7 +71,7 @@ solrAdminApp.controller('FilesController',
if ($scope.file && $scope.file !== '' && $scope.file.split('').pop()!=='/') {
var extension;
if ($scope.file === "managed-schema") {
- extension = contentTypeMap['xml'];
+ extension = 'xml';
} else {
extension = $scope.file.match( /\.(\w+)$/)[1] || '';
}