Simplify construction of appName from container name in JAASRealm.

There is no need to add a slash in front of the container name as
the method makeLegalForJAAS will strip it off directly afterwards.
That removes the need for a temporary variable.


git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@1837787 13f79535-47bb-0310-9956-ffa450edef68

java/org/apache/catalina/realm/JAASRealm.java

@@ -227,13 +227,7 @@ public void setContainer(Container container) {
         super.setContainer(container);
 
         if( appName==null  ) {
-            String name = container.getName();
-            if (!name.startsWith("/")) {
-                name = "/" + name;
-            }
-            name = makeLegalForJAAS(name);
-
-            appName=name;
+            appName = makeLegalForJAAS(container.getName());
 
             log.info("Set JAAS app name " + appName);
         }

webapps/docs/changelog.xml

@@ -106,6 +106,9 @@
       <fix>
         Use short circuit logic to prevent potential NPE in CorsFilter. (fschumacher)
       </fix>
+      <scode>
+        Simplify construction of appName from container name in JAASRealm. (fschumacher)
+      </scode>
     </changelog>
   </subsection>
   <subsection name="Coyote">