NihalJain commented on code in PR #6783:
URL: https://github.com/apache/hbase/pull/6783#discussion_r2028427274
##########
hbase-rest/src/test/java/org/apache/hadoop/hbase/rest/TestGetAndPutResource.java:
##########
@@ -321,6 +321,8 @@ public void testLatestCellGetJSON() throws IOException {
@Test
public void testURLEncodedKey() throws IOException, JAXBException {
+ // Requires UriCompliance.Violation.AMBIGUOUS_PATH_SEPARATOR
Review Comment:
In this test, request url resolves to
"/TestRowResource/http%3A%2F%2Fexample.com%2Ffoo/a:1" and is considered
ambiguous with Jetty 12 without the above setting. Basically we are having a
URL encoded string as row key here!
We will have to change the way we take input and maybe stop relying on
request path.
--
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: issues-unsubscr...@hbase.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
