Chat API {name=spaces/*} and unquoting issue about aiogoogle HOT 2 CLOSED

Nice catch. I should've opted for quote instead of quote_plus. As highlighted by the urllib docs:

By default, this function is intended for quoting the path section of a URL.

Also, according to the urllib docs, the only two differences between quote_plus and quote is that:

1. quote_plus replaces spaces with a + instead of a %20.
2. It does not treat forwards slashes as safe chars.

So, I suspect that changing to quote might cause future data inconsistency for some of the users of this lib. E.g. a user created a reaource using an encoded forward slash, but now can't fetch it because forward slashes are being escaped. That's assuming that Google APIs accept URL data in non idempotent requests, which i'd say is unlikely.

Anyway, I think it's best to change it to the correct quote function and bump the version up by one major version to give people a heads up about the change. What do you think?

Thanks for raising the issue!

from aiogoogle.

Sounds good to me! 👍
Thanks.

from aiogoogle.