Skip to content

Conversation

arcoraven
Copy link
Contributor

Problem solved

Fixing a potential edge case where the response body is read twice, causing an unrelated throw:

Body has already been used. It can only be used once. Use tee() first if you need to read it twice.

@arcoraven arcoraven requested a review from a team November 17, 2023 23:48
Copy link

changeset-bot bot commented Nov 17, 2023

🦋 Changeset detected

Latest commit: df01256

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@thirdweb-dev/service-utils Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@arcoraven
Copy link
Contributor Author

/release-pr

@arcoraven
Copy link
Contributor Author

/release-pr

@arcoraven
Copy link
Contributor Author

/release-pr

@arcoraven arcoraven added this pull request to the merge queue Nov 18, 2023
Merged via the queue into main with commit 8ff87be Nov 18, 2023
@arcoraven arcoraven deleted the ph/reuseRespBody branch November 18, 2023 01:55
@github-actions github-actions bot mentioned this pull request Nov 18, 2023
jnsdls pushed a commit that referenced this pull request Jun 19, 2024
* Fix EmbeddedWallet OTP UI

* use stable version
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.

1 participant