JinwooHwang opened a new pull request, #8041: URL: https://github.com/apache/geode/pull/8041
When a region-detail request names a path that does not resolve, the region services return "Region [<path>] is not available" in the errorOnRegion field, which the Pulse UI displays. Paths containing characters such as '<' or '&' did not display correctly. Encode the path with StringEscapeUtils.escapeHtml4 when building the message, so it displays as written. commons-text is already a compile dependency of geode-pulse. Region lookup is unchanged and continues to use the path as supplied. Adds RegionErrorMessageEncodingTest and RegionDetailErrorMessageIntegrationTest. <!-- Thank you for submitting a contribution to Apache Geode. --> <!-- In order to streamline review of your contribution we ask that you ensure you've taken the following steps. --> ### For all changes, please confirm: - [x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message? - [x] Has your PR been rebased against the latest commit within the target branch (typically `develop`)? - [x] Is your initial contribution a single, squashed commit? - [x] Does `gradlew build` run cleanly? - [ ] Have you written or updated unit tests to verify your changes? - [ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under [ASF 2.0](http://www.apache.org/legal/resolved.html#category-a)? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
