Addressed review comments

Author: ydodeja365

hbase-http/src/main/java/org/apache/hadoop/hbase/http/HttpServer.java

@@ -154,7 +154,7 @@ public class HttpServer implements FilterContainer {
   public static final String SPNEGO_PROXYUSER_FILTER = "SpnegoProxyUserFilter";
   public static final String NO_CACHE_FILTER = "NoCacheFilter";
   public static final String APP_DIR = "webapps";
-  public static final String HTTP_UI_SHOW_STACKTRACE_KEY = "hbase.ui.showStackTraces";
+  public static final String HTTP_UI_SHOW_STACKTRACE_KEY = "hbase.ui.show-stack-traces";
 
   public static final String METRIC_SERVLETS_CONF_KEY = "hbase.http.metrics.servlets";
   public static final String[] METRICS_SERVLETS_DEFAULT = { "jmx", "metrics", "prometheus" };
@@ -655,9 +655,8 @@ private void initializeWebServer(String name, String hostName, Configuration con
       }
     }
     // Check if disable stack trace property is configured
-    if (null != conf.get(HTTP_UI_SHOW_STACKTRACE_KEY, null)) {
-      // Get the configured value
-      boolean showStackTraces = conf.getBoolean(HTTP_UI_SHOW_STACKTRACE_KEY, false);
+    boolean showStackTraces = conf.getBoolean(HTTP_UI_SHOW_STACKTRACE_KEY, false);
+    if (showStackTraces)) {
       // Disable stack traces for server errors in UI
       webServer.setErrorHandler(new ErrorHandler());
       webServer.getErrorHandler().setShowStacks(showStackTraces);

src/main/asciidoc/_chapters/security.adoc

@@ -207,7 +207,7 @@ To disable showing stack traces in HBase UI for hiding sensitive information, se
 [source,xml]
 ----
 <property>
-  <name>hbase.ui.showStackTraces</name>
+  <name>hbase.ui.show-stack-traces</name>
   <value>false</value>
 </property>
 ----