Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Support for GET requests #81

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Oct 4, 2022
Merged

Support for GET requests #81

merged 4 commits into from
Oct 4, 2022

Conversation

kevin-carroll
Copy link
Member

@kevin-carroll kevin-carroll commented Sep 18, 2022 •
edited
Loading

Get Request Support

  • Added the ability to accept GET requests and query string through the default HTTP processor
    • Accepts query string parameters named: query, variables, operationName defined as constants in Constants.cs
  • The default behavior for parsing an HttpContext can be altered by inheriting from DefaultGraphQLHttpProcessor<TSchema> and overriding the ParseHttpContext() method.

Related: #80

@kevin-carroll kevin-carroll self-assigned this Sep 18, 2022
pmoleri
pmoleri reviewed Sep 19, 2022
View reviewed changes
src/graphql-aspnet/Defaults/DefaultGraphQLHttpProcessor{TSchema}.cs Outdated Show resolved Hide resolved
@kevin-carroll kevin-carroll linked an issue Sep 25, 2022 that may be closed by this pull request
HTTP GET support and query url param #80
Closed
@kevin-carroll kevin-carroll merged commit 0641023 into master Oct 4, 2022
@kevin-carroll kevin-carroll deleted the feature/getRequestSupport-80 branch October 4, 2022 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pmoleri pmoleri pmoleri left review comments

@graphql-aspnet-owner graphql-aspnet-owner graphql-aspnet-owner approved these changes

Assignees

@kevin-carroll kevin-carroll

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

HTTP GET support and query url param
3 participants
@kevin-carroll @pmoleri @graphql-aspnet-owner
Morty Proxy This is a proxified and sanitized view of the page, visit original site.