Merge pull request #19 from cyberark/fix-slash-escaping

Fix slash escaping

CHANGELOG.md

@@ -4,11 +4,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
-## [Unreleased]
+## [0.0.3]
 
 ### Fixed
 
 - Fixed application of conjurrc overrides of `Client` initialization params [#14](https://github.com/cyberark/conjur-api-python3/issues/14)
+- Fixed escaping of `/` in parameters of URL
 
 ## [0.0.2] - 2019-05-17
 
@@ -25,6 +26,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 The first tagged version.
 
-[Unreleased]: https://github.com/conjurinc/conjur-api-python3/compare/v0.0.2...HEAD
+[Unreleased]: https://github.com/conjurinc/conjur-api-python3/compare/v0.0.3...HEAD
+[0.0.3]: https://github.com/conjurinc/conjur-api-python3/compare/v0.0.2...0.0.3
 [0.0.2]: https://github.com/conjurinc/conjur-api-python3/compare/v0.0.1...0.0.2
 [0.0.1]: https://github.com/cyberark/conjur-api-python3/tree/v0.0.1

conjur/http.py

@@ -40,7 +40,7 @@ def invoke_endpoint(http_verb, endpoint, params, *args, check_errors=True,
         if key == 'url':
             params[key] = value
             continue
-        params[key] = quote(value)
+        params[key] = quote(value, safe='')
 
     url = endpoint.value.format(**params)
 

test/test_http.py

@@ -25,7 +25,7 @@ class HttpInvokeEndpointTest(unittest.TestCase):
     }
 
     ESCAPED_PARAMS = [
-        'abc/%24%21%40%23%24%25%5E%26%2A%28%29%20%5C%5B%5D%7B%7D',
+        'abc%2F%24%21%40%23%24%25%5E%26%2A%28%29%20%5C%5B%5D%7B%7D',
         '%29%28%2A%26%5E%25%24%23%40%21%5B%5D%7B%7D%20%3C%3E%3F',
     ]