Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1898: /varz page doesn't HTML-escape flag values ......................................................................
KUDU-1898: /varz page doesn't HTML-escape flag values This fixes the flag values are not properly HTML-escaped in the /varz web page. Verfied the flag values will be HTML-escaped in WebserverTest.TestRedactFlagsDump. Also I manually validated that the /varz page output is as expected. Change-Id: I64ebb0befe6bacb0fc90d50a1e34870656041a01 Reviewed-on: http://gerrit.cloudera.org:8080/6173 Tested-by: Kudu Jenkins Reviewed-by: Todd Lipcon <[email protected]> --- M src/kudu/server/default-path-handlers.cc M src/kudu/server/webserver-test.cc M src/kudu/util/flag_tags-test.cc M src/kudu/util/flags.cc M src/kudu/util/flags.h 5 files changed, 47 insertions(+), 24 deletions(-) Approvals: Todd Lipcon: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/6173 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I64ebb0befe6bacb0fc90d50a1e34870656041a01 Gerrit-PatchSet: 8 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Hao Hao <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
