Update client.js to capture all cookies from response

appwrite / sdk-for-cli

[READ-ONLY] Official Appwrite CLI >_

BSD 3-Clause "New" or "Revised" License

85 stars 29 forks source link

Update client.js to capture all cookies from response #104

Closed StarZeus closed 3 months ago

StarZeus commented 5 months ago

What does this PR do?

This PR enables appwrite-cli to capture all cookies in 'set-cookie' response headers

Test Plan

I have tested this change hosting appwrite in kubernetes environment where nginx-ingress controller adds its own cookies. And I was able to execute multiple appwrite cli commands without loggin in for each command.

Related PRs and Issues

https://github.com/appwrite/sdk-for-cli/issues/110

Have you read the Contributing Guidelines on issues?

Yes

stnguyen90 commented 4 months ago

Thanks for raising this PR! 🙏🏼 Unfortunately, this is a read-only repo as the code is generated by our SDK Generator, and changes made only here would be wiped in the next release. Would you be able to make the change in https://github.com/appwrite/sdk-generator/blob/master/templates/cli/lib/client.js.twig?