Author: todd
Date: Fri May 20 03:44:23 2011
New Revision: 1125222

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

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

Modified: hadoop/mapreduce/branches/branch-0.22/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hadoop/mapreduce/branches/branch-0.22/CHANGES.txt?rev=1125222&r1=1125221&r2=1125222&view=diff
==============================================================================
--- hadoop/mapreduce/branches/branch-0.22/CHANGES.txt (original)
+++ hadoop/mapreduce/branches/branch-0.22/CHANGES.txt Fri May 20 03:44:23 2011
@@ -217,6 +217,9 @@ Release 0.22.0 - Unreleased
     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-1354. Enhancements to JobTracker for better performance and

Modified: hadoop/mapreduce/branches/branch-0.22/src/java/mapred-default.xml
URL: 
http://svn.apache.org/viewvc/hadoop/mapreduce/branches/branch-0.22/src/java/mapred-default.xml?rev=1125222&r1=1125221&r2=1125222&view=diff
==============================================================================
--- hadoop/mapreduce/branches/branch-0.22/src/java/mapred-default.xml (original)
+++ hadoop/mapreduce/branches/branch-0.22/src/java/mapred-default.xml Fri May 
20 03:44:23 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/branches/branch-0.22/src/java/org/apache/hadoop/mapred/JSPUtil.java
URL: 
http://svn.apache.org/viewvc/hadoop/mapreduce/branches/branch-0.22/src/java/org/apache/hadoop/mapred/JSPUtil.java?rev=1125222&r1=1125221&r2=1125222&view=diff
==============================================================================
--- 
hadoop/mapreduce/branches/branch-0.22/src/java/org/apache/hadoop/mapred/JSPUtil.java
 (original)
+++ 
hadoop/mapreduce/branches/branch-0.22/src/java/org/apache/hadoop/mapred/JSPUtil.java
 Fri May 20 03:44:23 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/branches/branch-0.22/src/java/org/apache/hadoop/mapreduce/server/jobtracker/JTConfig.java
URL: 
http://svn.apache.org/viewvc/hadoop/mapreduce/branches/branch-0.22/src/java/org/apache/hadoop/mapreduce/server/jobtracker/JTConfig.java?rev=1125222&r1=1125221&r2=1125222&view=diff
==============================================================================
--- 
hadoop/mapreduce/branches/branch-0.22/src/java/org/apache/hadoop/mapreduce/server/jobtracker/JTConfig.java
 (original)
+++ 
hadoop/mapreduce/branches/branch-0.22/src/java/org/apache/hadoop/mapreduce/server/jobtracker/JTConfig.java
 Fri May 20 03:44:23 2011
@@ -104,4 +104,6 @@ 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";
 }

Modified: 
hadoop/mapreduce/branches/branch-0.22/src/java/org/apache/hadoop/mapreduce/util/ConfigUtil.java
URL: 
http://svn.apache.org/viewvc/hadoop/mapreduce/branches/branch-0.22/src/java/org/apache/hadoop/mapreduce/util/ConfigUtil.java?rev=1125222&r1=1125221&r2=1125222&view=diff
==============================================================================
--- 
hadoop/mapreduce/branches/branch-0.22/src/java/org/apache/hadoop/mapreduce/util/ConfigUtil.java
 (original)
+++ 
hadoop/mapreduce/branches/branch-0.22/src/java/org/apache/hadoop/mapreduce/util/ConfigUtil.java
 Fri May 20 03:44:23 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/branches/branch-0.22/src/test/mapred/org/apache/hadoop/mapred/TestWebUIAuthorization.java
URL: 
http://svn.apache.org/viewvc/hadoop/mapreduce/branches/branch-0.22/src/test/mapred/org/apache/hadoop/mapred/TestWebUIAuthorization.java?rev=1125222&r1=1125221&r2=1125222&view=diff
==============================================================================
--- 
hadoop/mapreduce/branches/branch-0.22/src/test/mapred/org/apache/hadoop/mapred/TestWebUIAuthorization.java
 (original)
+++ 
hadoop/mapreduce/branches/branch-0.22/src/test/mapred/org/apache/hadoop/mapred/TestWebUIAuthorization.java
 Fri May 20 03:44:23 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);


Reply via email to