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

fix: GraphQL over HTTP compliance (v3) #1685

Merged
merged 14 commits into from
Sep 5, 2022
Merged

fix: GraphQL over HTTP compliance (v3) #1685

merged 14 commits into from
Sep 5, 2022

Conversation

enisdenjo
Copy link
Collaborator

Using audits from graphql-http to check for GraphQL over HTTP spec compliance.

@changeset-bot
Copy link

changeset-bot bot commented Aug 30, 2022

⚠️ No Changeset found

Latest commit: d99e3f5

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

💥 An error occurred when fetching the changed packages and changesets in this PR
Some errors occurred when validating the changesets config:
The package or glob expression "*-example" is specified in the `ignore` option but it is not found in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.

@enisdenjo enisdenjo self-assigned this Aug 30, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Aug 30, 2022

✅ Benchmark Results

     ✓ no_errors
     ✓ expected_result

     checks.........................: 100.00% ✓ 199514      ✗ 0    
     data_received..................: 30 MB   991 kB/s
     data_sent......................: 12 MB   382 kB/s
     http_req_blocked...............: avg=1.15µs   min=700ns   med=1µs      max=4.92ms   p(90)=1.4µs    p(95)=1.7µs   
     http_req_connecting............: avg=1ns      min=0s      med=0s       max=151.89µs p(90)=0s       p(95)=0s      
   ✓ http_req_duration..............: avg=232.48µs min=160.7µs med=212.39µs max=24.61ms  p(90)=258.63µs p(95)=286.79µs
       { expected_response:true }...: avg=232.48µs min=160.7µs med=212.39µs max=24.61ms  p(90)=258.63µs p(95)=286.79µs
     http_req_failed................: 0.00%   ✓ 0           ✗ 99757
     http_req_receiving.............: avg=17.52µs  min=10µs    med=16.2µs   max=8.06ms   p(90)=20.6µs   p(95)=24.5µs  
     http_req_sending...............: avg=5.84µs   min=3.9µs   med=4.59µs   max=4.79ms   p(90)=6.3µs    p(95)=9.19µs  
     http_req_tls_handshaking.......: avg=0s       min=0s      med=0s       max=0s       p(90)=0s       p(95)=0s      
     http_req_waiting...............: avg=209.12µs min=139.8µs med=191.09µs max=24.25ms  p(90)=231.09µs p(95)=255.89µs
     http_reqs......................: 99757   3325.102325/s
     iteration_duration.............: avg=296.31µs min=212.2µs med=274.19µs max=24.83ms  p(90)=330.09µs p(95)=364.71µs
     iterations.....................: 99757   3325.102325/s
     vus............................: 0       min=0         max=1  
     vus_max........................: 1       min=1         max=1  

@n1ru4l n1ru4l merged commit fd11462 into v3 Sep 5, 2022
@n1ru4l n1ru4l deleted the v3-graphql-over-http branch September 5, 2022 09:47
@charlypoly
Copy link
Contributor

/theguild newsletter

This was referenced Sep 21, 2022
@saihaj saihaj mentioned this pull request Oct 17, 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.

None yet

5 participants