fix: Replace string-template with url-template #1108
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #1099
Path parameters are incorrectly encoded. This fix uses a different module to parse the path parameters, using the same RFC6570 format as the API Discovery Documents uses.
From the [API Discovery Service Documentation]:(https://developers.google.com/discovery/v1/using#build-compose):
In
google-api-nodejs-client
the special URI Template operators (+
,/
and*
) were stripped from the path, to make the templates compatible with thestring-template
module.The
string-template
module should be replaced with a URI Template module.npm test
succeedsapis/