You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by de...@apache.org on 2015/11/12 21:01:41 UTC
svn commit: r1714109 -
/uima/sandbox/uima-ducc/trunk/uima-ducc-web/src/main/java/org/apache/uima/ducc/ws/authentication/DuccAsUser.java
Author: degenaro
Date: Thu Nov 12 20:01:41 2015
New Revision: 1714109
URL: http://svn.apache.org/viewvc?rev=1714109&view=rev
Log:
UIMA-4690 fix DuccAsUser (webserver)
Modified:
uima/sandbox/uima-ducc/trunk/uima-ducc-web/src/main/java/org/apache/uima/ducc/ws/authentication/DuccAsUser.java
Modified: uima/sandbox/uima-ducc/trunk/uima-ducc-web/src/main/java/org/apache/uima/ducc/ws/authentication/DuccAsUser.java
URL: http://svn.apache.org/viewvc/uima/sandbox/uima-ducc/trunk/uima-ducc-web/src/main/java/org/apache/uima/ducc/ws/authentication/DuccAsUser.java?rev=1714109&r1=1714108&r2=1714109&view=diff
==============================================================================
--- uima/sandbox/uima-ducc/trunk/uima-ducc-web/src/main/java/org/apache/uima/ducc/ws/authentication/DuccAsUser.java (original)
+++ uima/sandbox/uima-ducc/trunk/uima-ducc-web/src/main/java/org/apache/uima/ducc/ws/authentication/DuccAsUser.java Thu Nov 12 20:01:41 2015
@@ -19,6 +19,7 @@
package org.apache.uima.ducc.ws.authentication;
import java.io.BufferedReader;
+import java.io.File;
import java.io.InputStreamReader;
import java.util.ArrayList;
import java.util.Map;
@@ -28,13 +29,14 @@ import org.apache.uima.ducc.common.utils
import org.apache.uima.ducc.common.utils.DuccPropertiesResolver;
import org.apache.uima.ducc.common.utils.Utils;
-
public class DuccAsUser {
private static DuccLogger duccLogger = DuccLoggerComponents.getWsLogger(DuccAsUser.class.getName());
public static String magicString = "1001 Command launching...";
+ private static File devNull = new File("/dev/null");
+
public static String duckling(String user, String[] args) {
String methodName = "duckling";
@@ -86,10 +88,10 @@ public class DuccAsUser {
}
try {
+ pb = pb.redirectError(devNull);
Process process = pb.start();
String line;
BufferedReader bri = new BufferedReader(new InputStreamReader(process.getInputStream()));
- BufferedReader bre = new BufferedReader(new InputStreamReader(process.getErrorStream()));
boolean trigger = false;
duccLogger.trace(methodName, null, "read stdout: start");
while ((line = bri.readLine()) != null) {
@@ -104,13 +106,6 @@ public class DuccAsUser {
}
bri.close();
duccLogger.trace(methodName, null, "read stdout: end");
- duccLogger.trace(methodName, null, "read stderr: start");
- while ((line = bre.readLine()) != null) {
- duccLogger.warn(methodName, null, "stderr: "+line);
- retVal.append(line);
- }
- bre.close();
- duccLogger.trace(methodName, null, "read stderr: end");
duccLogger.trace(methodName, null, "process waitfor: start");
process.waitFor();
duccLogger.trace(methodName, null, "process waitfor: end");
@@ -164,10 +159,10 @@ public class DuccAsUser {
}
try {
+ pb = pb.redirectError(devNull);
Process process = pb.start();
String line;
BufferedReader bri = new BufferedReader(new InputStreamReader(process.getInputStream()));
- BufferedReader bre = new BufferedReader(new InputStreamReader(process.getErrorStream()));
boolean trigger = true;
while ((line = bri.readLine()) != null) {
if(trigger) {
@@ -178,10 +173,6 @@ public class DuccAsUser {
}
}
bri.close();
- while ((line = bre.readLine()) != null) {
- retVal.append(line);
- }
- bre.close();
process.waitFor();
}
catch(Exception e) {