Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Fix SignalR client code not reading responses correctly #2626

Merged
merged 2 commits into from
Nov 15, 2022
Merged

Conversation

Porges
Copy link
Member

@Porges Porges commented Nov 15, 2022

The Response object must have the JSON loaded via json() (backend.request was updated but this callsite was missed).

Also, import the Onefuzz type so that this is type-checked to avoid the same problem in future.

@Porges Porges changed the title Fix SignalR not reading responses correctly Fix SignalR client code not reading responses correctly Nov 15, 2022
@codecov-commenter
Copy link

codecov-commenter commented Nov 15, 2022

Codecov Report

Merging #2626 (dd5150b) into main (e546f78) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2626   +/-   ##
=======================================
  Coverage   29.37%   29.37%           
=======================================
  Files         290      290           
  Lines       35596    35596           
=======================================
  Hits        10455    10455           
  Misses      25141    25141           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Porges Porges enabled auto-merge (squash) November 15, 2022 02:02
@Porges Porges merged commit 07befc5 into main Nov 15, 2022
@nharper285 nharper285 mentioned this pull request Nov 28, 2022
@Porges Porges deleted the fix-signalr branch December 4, 2022 22:41
@ghost ghost locked as resolved and limited conversation to collaborators Jan 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants