Skip to content

Conversation

@rose-a
Copy link
Collaborator

@rose-a rose-a commented Mar 15, 2020

fixes #90

@rose-a
Copy link
Collaborator Author

rose-a commented Mar 20, 2020

I changed it to GraphQLHttpResponse<T> : GraphQLResponse<T>, IMO this is a really clean solution.

There is an extension method AsGraphQLHttpResponse() to cast the returned GraphQLResponse<T> back to a GraphQLHttpResponse<T> (necessary if a dynamic type was used) which will throw when the cast is not possible (aka using a websocket).

@sungam3r
Copy link
Member

Indeed, more clean.

@rose-a rose-a merged commit 525b915 into develop Mar 20, 2020
@rose-a rose-a deleted the access-to-http-response-headers branch March 26, 2020 10:30
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.

Retrieving Headers from Response

3 participants