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

Fixed GetAsync method in Rest class #10797

Merged
merged 3 commits into from
Oct 4, 2022

Conversation

Proton-V
Copy link
Contributor

Changes

Overview

Fixed problem with using request in Rest.cs class

Fixes

  • ProcessRequestAsync method in Rest.cs

@Proton-V
Copy link
Contributor Author

@keveleigh
Hello. I see this warning from azure.
I think you can approve these changes and pipeline will accept branch?

@microsoft microsoft deleted a comment from azure-pipelines bot Jul 29, 2022
@keveleigh
Copy link
Contributor

@keveleigh Hello. I see this warning from azure. I think you can approve these changes and pipeline will accept branch?

I'll pull this down and test it out today! Thanks!

@Zee2
Copy link
Contributor

Zee2 commented Aug 23, 2022

@keveleigh updates on this?

@keveleigh
Copy link
Contributor

Thanks for this fix! Worked great for me locally.
I pushed one slight change to BytesToString, as I was hitting a null ref in cases where the connection wasn't successful. This change allowed subsequent errors to surface for debugging purposes.

@keveleigh keveleigh self-assigned this Oct 4, 2022
@keveleigh keveleigh merged commit 3c09ff5 into microsoft:main Oct 4, 2022
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.

Utilities WebRequestRest Rest.GetAsync() Causes Unity To Hang
5 participants