Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/14573 )
Change subject: webserver: add support for Knox URL rewriting ...................................................................... Patch Set 5: (4 comments) Thanks for the review. http://gerrit.cloudera.org:8080/#/c/14573/5//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/14573/5//COMMIT_MSG@76 PS5, Line 76: But it's also fraught in that Impala : doesn't always know its own hostname > Fwiw, we basically get around this by expected that users may need to set s Our cross-server links are all IP-based rather than hostname-based. Do you think that's problematic? http://gerrit.cloudera.org:8080/#/c/14573/5/src/kudu/server/webserver.cc File src/kudu/server/webserver.cc: http://gerrit.cloudera.org:8080/#/c/14573/5/src/kudu/server/webserver.cc@750 PS5, Line 750: {{base_url}}{{path}} > '/'? {{path}} is prefixed with a '/'; see the various calls to Webserver::RegisterPathHandler and Webserver::RegisterPrerenderedPathHandler. http://gerrit.cloudera.org:8080/#/c/14573/5/www/home.mustache File www/home.mustache: http://gerrit.cloudera.org:8080/#/c/14573/5/www/home.mustache@22 PS5, Line 22: {{base_url}}{{path}} > should there be a '/' between these? {{path}} is prefixed with a '/'; see the various calls to Webserver::RegisterPathHandler and Webserver::RegisterPrerenderedPathHandler. http://gerrit.cloudera.org:8080/#/c/14573/5/www/tablets.mustache File www/tablets.mustache: http://gerrit.cloudera.org:8080/#/c/14573/5/www/tablets.mustache@51 PS5, Line 51: {{base_url}}{{url}} > '/'? {{url}} is prefixed with a '/': link_json["url"] = Substitute("/tablet?id=$0", UrlEncodeToString(status.tablet_id())); -- To view, visit http://gerrit.cloudera.org:8080/14573 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iee92cb094b81609356acf858b7c549b6c281a7e5 Gerrit-Change-Number: 14573 Gerrit-PatchSet: 5 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]> Gerrit-Reviewer: Tidy Bot (241) Gerrit-Comment-Date: Thu, 31 Oct 2019 19:28:49 +0000 Gerrit-HasComments: Yes
