You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tika.apache.org by ma...@apache.org on 2015/05/02 08:38:11 UTC

svn commit: r1677279 - /tika/trunk/tika-server/src/main/java/org/apache/tika/server/resource/TikaResource.java

Author: mattmann
Date: Sat May  2 06:38:10 2015
New Revision: 1677279

URL: http://svn.apache.org/r1677279
Log:
- fix for TIKA-1621: TikaResource should log errors determining ContentDisposition

Modified:
    tika/trunk/tika-server/src/main/java/org/apache/tika/server/resource/TikaResource.java

Modified: tika/trunk/tika-server/src/main/java/org/apache/tika/server/resource/TikaResource.java
URL: http://svn.apache.org/viewvc/tika/trunk/tika-server/src/main/java/org/apache/tika/server/resource/TikaResource.java?rev=1677279&r1=1677278&r2=1677279&view=diff
==============================================================================
--- tika/trunk/tika-server/src/main/java/org/apache/tika/server/resource/TikaResource.java (original)
+++ tika/trunk/tika-server/src/main/java/org/apache/tika/server/resource/TikaResource.java Sat May  2 06:38:10 2015
@@ -82,7 +82,7 @@ public class TikaResource {
     public static final String X_TIKA_PDF_HEADER_PREFIX = "X-Tika-PDF";
 
 
-    private final Log logger = LogFactory.getLog(TikaResource.class);
+    private static final Log logger = LogFactory.getLog(TikaResource.class);
 
     private TikaConfig tikaConfig;
 
@@ -131,6 +131,12 @@ public class TikaResource {
                 }
             } catch (ParseException e) {
                 // not a valid content-disposition field
+            	e.printStackTrace();
+            	logger.warn(String.format(
+                        Locale.ROOT,
+                        "Parse exception %s determining content disposition",
+                        e.getMessage()
+                ), e);
             }
         }