transport replaceVars url escape values

[odedniv]

Fixes #2895

According to https://cloud.google.com/storage/docs/json_api/#encoding URI paths parts are supposed to be encoded.

For example in google_storage_object_access_control if the object name contains a /, the request will fail with:

* google_storage_object_access_control.public_rule: Error creating ObjectAccessControl: googleapi: got HTTP response code 404 with body: Not Found

To my knowledge, replaceVars() is specifically for replacing variables in URLs (as for the name of the parameter "linkTmpl"), so I'm assuming this would work globally.

[odedniv]

It contains conflicts now because 966229a allowed adding % to variable names in order to URL encode them, but I believe it should always URL encode them because replaceVars is used specifically for URLs.

[odedniv]

Maybe closed by GoogleCloudPlatform/magic-modules#1717 (if I understand the repo structure correctly)

[odedniv]

Not relevant as discussed in GoogleCloudPlatform/magic-modules#1717

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!