Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: appwrite exception already read #1050

Merged
merged 4 commits into from
Mar 17, 2025
Merged

fix: appwrite exception already read #1050

merged 4 commits into from
Mar 17, 2025

Conversation

ChiragAgg5k
Copy link
Member

@ChiragAgg5k ChiragAgg5k commented Mar 14, 2025

What does this PR do?

A response stream can only be read once. That's why reading it for text and then reading it again for json gave error "Already read". The error was caught and fixed in web and node, but left in react-native. This PR makes the logic consistent across all three.

Test Plan

Related PRs and Issues

Have you read the Contributing Guidelines on issues?

Yes.

@ChiragAgg5k ChiragAgg5k requested a review from stnguyen90 March 15, 2025 16:04
@stnguyen90 stnguyen90 merged commit a08f53a into master Mar 17, 2025
37 checks passed
@stnguyen90 stnguyen90 deleted the fix-already-read branch March 17, 2025 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants