Bug in escape function defined in utils.cc
In the file https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-cpp-lib/-/blob/master/src/src/lib/shared/utils.cc, line 264 there's escape function defined. It contains a bug in line 270:
if (c < 0 && (isalnum(c) || c == '-' || c == '_' || c == '.' || c == '~'))
Specifically, the "if c < 0" condition shouldn't be there, at least from my point of view. As a result, the following code of sdapi library:
http.set_url(_sdmanager->getSDUrl() +
"/dataset/tenant/" + escape(tenant) +
"/subproject/" + escape(subproject) +
"/exist");
produces the following URL:
"https://osdu-ship.msft-osdu-test.org/seistore-svc/api/v3/dataset/tenant/%6F%70%65%6E%64%65%73/subproject/%6D%69%63%68%61%65%6C%76%31/exist"
instead of
"https://osdu-ship.msft-osdu-test.org/seistore-svc/api/v3/dataset/tenant/opendes/subproject/michaelv1/exist"