Skip to content

feat(graphql): support http get and post json request#6343

Merged
spacewander merged 4 commits into
apache:masterfrom
shuaijinchao:graphql/support/get/json
Feb 18, 2022
Merged

feat(graphql): support http get and post json request#6343
spacewander merged 4 commits into
apache:masterfrom
shuaijinchao:graphql/support/get/json

Conversation

@shuaijinchao

@shuaijinchao shuaijinchao commented Feb 16, 2022

Copy link
Copy Markdown
Member

What this PR does / why we need it:

FIX #6266

Pre-submission checklist:

  • Did you explain what problem does this PR solve? Or what new features have been added?
  • Have you added corresponding test cases?
  • Have you modified the corresponding document?
  • Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

Comment thread apisix/core/ctx.lua Outdated
return nil, "failed to read graphql data, " .. (err or "request body has zero size")
end

if request.header(nil, "Content-Type") == GRAPHQL_REQ_MIME_JSON then

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can pass ctx here?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Of course, I update it later.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

Comment thread apisix/core/ctx.lua
@shuaijinchao
shuaijinchao marked this pull request as ready for review February 17, 2022 08:27
@spacewander
spacewander merged commit 00b7b01 into apache:master Feb 18, 2022
tzssangglass pushed a commit to tzssangglass/apisix that referenced this pull request Feb 21, 2022
hongbinhsu pushed a commit to fitphp/apix that referenced this pull request Feb 23, 2022
* upstream: (52 commits)
  feat: add kubernetes discovery module (apache#4880)
  docs: fix For L7 proxy -> For L4 proxy (apache#6423)
  fix(deps): upgrade jsonschema to 0.9.8 (apache#6407)
  docs: translate Chinese to English in en clickhouse-logger (apache#6416)
  docs: add zh proxy-control.md&modify other doc error (apache#6346)
  docs: update public API relative usage (apache#6318)
  docs(cn): remove datadog from sidebar & fix doc lint conf (apache#6411)
  fix(request-validation): should not limit the urlencoded post args number (apache#6396)
  docs: fix configuration file typo (apache#6395)
  docs(extern-plugin): the implementation of runner (apache#6336)
  docs: polishing skywalking-logger plugin's docs (apache#6377)
  doc: adjust the directory structure of observability's documents (apache#6391)
  change(admin): empty nodes should be encoded as array (apache#6384)
  fix: should not limit the header number (apache#6379)
  ci: remove unnecessary tmate action (apache#6367)
  fix(opentelemetry): batch_span_processor export zero length spans (apache#6349)
  feat(graphql): support http get and post json request (apache#6343)
  feat: support for configuring the number of etcd health check retries (apache#6322)
  feat(wasm): support getting request body (apache#6325)
  fix(hmac-auth): hmac-auth plugin sort array param (apache#6314)
  ...
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.

bug: ctx.lua#59 parse_graphql(ctx)

3 participants