szaszm commented on code in PR #1362:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1362#discussion_r911902756
##########
libminifi/include/utils/HTTPUtils.h:
##########
@@ -31,14 +31,14 @@ so we convert localhost to our local hostname.
inline bool parse_http_components(const std::string &url, std::string &port,
std::string &scheme, std::string &path) {
#ifdef WIN32
auto hostname =
(url.find(org::apache::nifi::minifi::io::Socket::getMyHostName()) !=
std::string::npos ? org::apache::nifi::minifi::io::Socket::getMyHostName() :
"localhost");
- std::string regex_str = "^(http|https)://(" + hostname + ":)([0-9]+)?(/.*)$";
+ std::string regex_str = "(http|https)://(" + hostname + ":)([0-9]+)?(/.*)$";
#else
- std::string regex_str = "^(http|https)://(localhost:)([0-9]+)?(/.*)$";
+ std::string regex_str = "(http|https)://(localhost:)([0-9]+)?(/.*)$";
Review Comment:
Please remove the trailing `$` sign / end of line match as well. It should
no longer be necessary, just like `^`.
--
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...@nifi.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
