Pyrebase icon indicating copy to clipboard operation
Pyrebase copied to clipboard

Fix erroneous URL encoding of query parameters

Open tejasvi opened this issue 4 years ago • 2 comments

quote is redundant since we anyway urlencode(parameters) at the end.

tejasvi avatar Aug 05 '21 04:08 tejasvi

I didn't see that there was someone who had this same problem. #393

I tried your solution and it works.

AlfonsoAgAr avatar Sep 16 '21 17:09 AlfonsoAgAr

Actually the double encoding relies on previously buggy behavior of REST API which decodes twice. https://github.com/firebase/firebase-admin-python/issues/189. State of python client is dismal. Admin SDK has various bug fixes and perf improvements while only pyrebase supports client authentication. Looks like I'll be migrating to Node.js in future, though PubNub looks promising for now.

tejasvi avatar Oct 10 '21 14:10 tejasvi