Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6173
to look at the new patch set (#5).
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
---
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, 50 insertions(+), 24 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/73/6173/5
--
To view, visit http://gerrit.cloudera.org:8080/6173
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I64ebb0befe6bacb0fc90d50a1e34870656041a01
Gerrit-PatchSet: 5
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
