You are viewing a plain text version of this content. The canonical link for it is here.
Posted to mapreduce-commits@hadoop.apache.org by to...@apache.org on 2011/05/20 05:45:11 UTC

svn commit: r1125223 - in /hadoop/mapreduce/trunk: ./ src/java/ src/java/org/apache/hadoop/mapred/ src/java/org/apache/hadoop/mapreduce/server/jobtracker/ src/java/org/apache/hadoop/mapreduce/util/ src/test/mapred/org/apache/hadoop/mapred/

Author: todd
Date: Fri May 20 03:45:11 2011
New Revision: 1125223

URL: http://svn.apache.org/viewvc?rev=1125223&view=rev
Log:
MAPREDUCE-2516. Rename webinterface.private.actions to mapreduce.jobtracker.webinterface.trusted. Contributed by Ari Rabkin.

Modified:
    hadoop/mapreduce/trunk/CHANGES.txt
    hadoop/mapreduce/trunk/src/java/mapred-default.xml
    hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapred/JSPUtil.java
    hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/server/jobtracker/JTConfig.java
    hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/util/ConfigUtil.java
    hadoop/mapreduce/trunk/src/test/mapred/org/apache/hadoop/mapred/TestWebUIAuthorization.java

Modified: hadoop/mapreduce/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/trunk/CHANGES.txt?rev=1125223&r1=1125222&r2=1125223&view=diff
==============================================================================
--- hadoop/mapreduce/trunk/CHANGES.txt (original)
+++ hadoop/mapreduce/trunk/CHANGES.txt Fri May 20 03:45:11 2011
@@ -119,6 +119,9 @@ Trunk (unreleased changes)
     MAPREDUCE-2372. TaskLogAppender mechanism shouldn't be set up in
     log4j.properties (todd)
 
+    MAPREDUCE-2516. Rename webinterface.private.actions to
+    mapreduce.jobtracker.webinterface.trusted (Ari Rabkin via todd)
+
   OPTIMIZATIONS
     
     MAPREDUCE-2026. Make JobTracker.getJobCounters() and

Modified: hadoop/mapreduce/trunk/src/java/mapred-default.xml
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/trunk/src/java/mapred-default.xml?rev=1125223&r1=1125222&r2=1125223&view=diff
==============================================================================
--- hadoop/mapreduce/trunk/src/java/mapred-default.xml (original)
+++ hadoop/mapreduce/trunk/src/java/mapred-default.xml Fri May 20 03:45:11 2011
@@ -1004,6 +1004,16 @@
 </property>
 
 <property>
+  <name>mapreduce.jobtracker.webinterface.trusted</name>
+  <value>false</value>
+  <description> If set to true, the web interface of the JobTracker                       
+                will include actions such as kill job that are security sensitive.
+                Leave this option as false if untrusted users have access to the web interface.                              
+  </description>
+</property>
+
+
+<property>
   <name>mapreduce.tasktracker.indexcache.mb</name>
   <value>10</value>
   <description> The maximum memory that a task tracker allows for the 

Modified: hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapred/JSPUtil.java
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapred/JSPUtil.java?rev=1125223&r1=1125222&r2=1125223&view=diff
==============================================================================
--- hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapred/JSPUtil.java (original)
+++ hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapred/JSPUtil.java Fri May 20 03:45:11 2011
@@ -49,7 +49,6 @@ import org.apache.hadoop.util.ServletUti
 import org.apache.hadoop.util.StringUtils;
 
 class JSPUtil {
-  static final String PRIVATE_ACTIONS_KEY = "webinterface.private.actions";
 
   //LRU based cache
   private static final Map<String, JobInfo> jobHistoryCache = 
@@ -476,7 +475,7 @@ class JSPUtil {
   }
 
   static boolean privateActionsAllowed(JobConf conf) {
-    return conf.getBoolean(PRIVATE_ACTIONS_KEY, false);
+    return conf.getBoolean(JTConfig.PRIVATE_ACTIONS_KEY, false);
   }
 
   static Path getJobConfFilePath(Path logFile) {

Modified: hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/server/jobtracker/JTConfig.java
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/server/jobtracker/JTConfig.java?rev=1125223&r1=1125222&r2=1125223&view=diff
==============================================================================
--- hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/server/jobtracker/JTConfig.java (original)
+++ hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/server/jobtracker/JTConfig.java Fri May 20 03:45:11 2011
@@ -107,6 +107,8 @@ public interface JTConfig extends MRConf
   public static final String JT_USER_NAME = "mapreduce.jobtracker.kerberos.principal";
   public static final String JT_KEYTAB_FILE = 
     "mapreduce.jobtracker.keytab.file";
+  public static final String PRIVATE_ACTIONS_KEY = 
+     "mapreduce.jobtracker.webinterface.trusted";
   public static final String JT_PLUGINS = 
     "mapreduce.jobtracker.plugins";
 }

Modified: hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/util/ConfigUtil.java
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/util/ConfigUtil.java?rev=1125223&r1=1125222&r2=1125223&view=diff
==============================================================================
--- hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/util/ConfigUtil.java (original)
+++ hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/util/ConfigUtil.java Fri May 20 03:45:11 2011
@@ -509,6 +509,9 @@ public class ConfigUtil {
       new String[] {org.apache.hadoop.mapred.pipes.Submitter.PARTITIONER});
     Configuration.addDeprecation("mapred.pipes.user.inputformat", 
       new String[] {org.apache.hadoop.mapred.pipes.Submitter.INPUT_FORMAT});
+    
+    Configuration.addDeprecation("webinterface.private.actions", 
+        new String[]{JTConfig.PRIVATE_ACTIONS_KEY});
   }
 }
 

Modified: hadoop/mapreduce/trunk/src/test/mapred/org/apache/hadoop/mapred/TestWebUIAuthorization.java
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/trunk/src/test/mapred/org/apache/hadoop/mapred/TestWebUIAuthorization.java?rev=1125223&r1=1125222&r2=1125223&view=diff
==============================================================================
--- hadoop/mapreduce/trunk/src/test/mapred/org/apache/hadoop/mapred/TestWebUIAuthorization.java (original)
+++ hadoop/mapreduce/trunk/src/test/mapred/org/apache/hadoop/mapred/TestWebUIAuthorization.java Fri May 20 03:45:11 2011
@@ -674,7 +674,7 @@ public class TestWebUIAuthorization exte
 
     props.setProperty("dfs.permissions.enabled", "false");
     
-    props.setProperty(JSPUtil.PRIVATE_ACTIONS_KEY, "true");
+    props.setProperty(JTConfig.PRIVATE_ACTIONS_KEY, "true");
     props.setProperty(MRJobConfig.SETUP_CLEANUP_NEEDED, "false");
     props.setProperty(MRConfig.MR_SUPERGROUP, "superGroup");
     props.setProperty(MRConfig.MR_ADMINS, mrAdminUser + " " + mrAdminGroup);