Add ability to not fail when pong is not received. #2815
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
I also changed how the read deadline set works a little, the reason for this is that the protocol allows for pong to be sent without a ping. So setting a read deadline on receiving pong isn't great. Instead we should always set the read deadline on sending ping. Though to do this we need to know whether we have received a pong or not. Because if we set the read deadline when the previous ping still hasn't received the pong. Then it will never hit the deadline.
|
Thanks! I appreciate you giving more attention to the websocket support in particular, as I'm not actively using it in production at the moment. I noticed that in this PR, although it has good comments, it doesn't actually mention the change anywhere in the docs. Do you mind looking over https://github.com/99designs/gqlgen/blob/master/docs/content/config.md and the other documentation that might be good to update for this and gqlgen's websocket support in general? |
nicolerenee
referenced
this pull request
in infratographer/metadata-api
Oct 20, 2023
[](https://renovatebot.com) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [github.com/99designs/gqlgen](https://togithub.com/99designs/gqlgen) | require | patch | `v0.17.38` -> `v0.17.39` | --- ### Release Notes <details> <summary>99designs/gqlgen (github.com/99designs/gqlgen)</summary> ### [`v0.17.39`](https://togithub.com/99designs/gqlgen/releases/tag/v0.17.39) [Compare Source](https://togithub.com/99designs/gqlgen/compare/v0.17.38...v0.17.39) #### What's Changed - Breaking API: Allow WebsocketInitFunc to add payload to Ack ([#​4](https://togithub.com/99designs/gqlgen/issues/4)) by [@​telemenar](https://togithub.com/telemenar) in [https://github.com/99designs/gqlgen/pull/2791](https://togithub.com/99designs/gqlgen/pull/2791) - add close flag into wsConnection to avoid duplicate calls of CloseFunc by [@​vlad-tokarev](https://togithub.com/vlad-tokarev) in [https://github.com/99designs/gqlgen/pull/2803](https://togithub.com/99designs/gqlgen/pull/2803) - fix: CodeGen for omit_slice_element_pointers and GetMany Entity Resolvers by [@​parkerroan](https://togithub.com/parkerroan) in [https://github.com/99designs/gqlgen/pull/2802](https://togithub.com/99designs/gqlgen/pull/2802) - feat: update getting-started CreateTodo mutationResolver by [@​gitxiongpan](https://togithub.com/gitxiongpan) in [https://github.com/99designs/gqlgen/pull/2810](https://togithub.com/99designs/gqlgen/pull/2810) - Feature: Support Apollo Federation Auth Directives by [@​parkerroan](https://togithub.com/parkerroan) in [https://github.com/99designs/gqlgen/pull/2809](https://togithub.com/99designs/gqlgen/pull/2809) - Consider go type name when autobinding by [@​dany74q](https://togithub.com/dany74q) in [https://github.com/99designs/gqlgen/pull/2812](https://togithub.com/99designs/gqlgen/pull/2812) - Update generated files that change when building by [@​telemenar](https://togithub.com/telemenar) in [https://github.com/99designs/gqlgen/pull/2813](https://togithub.com/99designs/gqlgen/pull/2813) - Add a pong only keep alive for the new protocol by [@​telemenar](https://togithub.com/telemenar) in [https://github.com/99designs/gqlgen/pull/2814](https://togithub.com/99designs/gqlgen/pull/2814) - Store parsed Schema on ExecutableSchema rather than use global variable by [@​gitxiongpan](https://togithub.com/gitxiongpan) in [https://github.com/99designs/gqlgen/pull/2811](https://togithub.com/99designs/gqlgen/pull/2811) - Add ability to not fail when pong is not received. by [@​telemenar](https://togithub.com/telemenar) in [https://github.com/99designs/gqlgen/pull/2815](https://togithub.com/99designs/gqlgen/pull/2815) - Adding duration scalar conforming to ISO8601 standard by [@​rwrz](https://togithub.com/rwrz) in [https://github.com/99designs/gqlgen/pull/2800](https://togithub.com/99designs/gqlgen/pull/2800) - Bump postcss from 8.4.24 to 8.4.31 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2819](https://togithub.com/99designs/gqlgen/pull/2819) - Add [@​interfaceObject](https://togithub.com/interfaceObject) and [@​composeDirective](https://togithub.com/composeDirective) at Federation 2 directive lists. by [@​wangmir](https://togithub.com/wangmir) in [https://github.com/99designs/gqlgen/pull/2821](https://togithub.com/99designs/gqlgen/pull/2821) #### New Contributors - [@​vlad-tokarev](https://togithub.com/vlad-tokarev) made their first contribution in [https://github.com/99designs/gqlgen/pull/2803](https://togithub.com/99designs/gqlgen/pull/2803) - [@​parkerroan](https://togithub.com/parkerroan) made their first contribution in [https://github.com/99designs/gqlgen/pull/2802](https://togithub.com/99designs/gqlgen/pull/2802) - [@​dany74q](https://togithub.com/dany74q) made their first contribution in [https://github.com/99designs/gqlgen/pull/2812](https://togithub.com/99designs/gqlgen/pull/2812) - [@​rwrz](https://togithub.com/rwrz) made their first contribution in [https://github.com/99designs/gqlgen/pull/2800](https://togithub.com/99designs/gqlgen/pull/2800) - [@​wangmir](https://togithub.com/wangmir) made their first contribution in [https://github.com/99designs/gqlgen/pull/2821](https://togithub.com/99designs/gqlgen/pull/2821) **Full Changelog**: 99designs/gqlgen@v0.17.38...v0.17.39 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Enabled. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/infratographer/metadata-api). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi44My4wIiwidXBkYXRlZEluVmVyIjoiMzcuMTkuMiIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
](https://renovatebot.com){kind=link}
github-merge-queue bot
referenced
this pull request
in infratographer/x
Apr 2, 2024
[](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [github.com/99designs/gqlgen](https://togithub.com/99designs/gqlgen) | `v0.17.38` -> `v0.17.45` | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>99designs/gqlgen (github.com/99designs/gqlgen)</summary> ### [`v0.17.45`](https://togithub.com/99designs/gqlgen/releases/tag/v0.17.45) [Compare Source](https://togithub.com/99designs/gqlgen/compare/v0.17.44...v0.17.45) #### What's Changed - Bump github.com/matryer/moq from 0.3.3 to 0.3.4 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2939](https://togithub.com/99designs/gqlgen/pull/2939) - Bump [@​apollo/client](https://togithub.com/apollo/client) from 3.9.4 to 3.9.5 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2938](https://togithub.com/99designs/gqlgen/pull/2938) - Bump vitest from 1.2.2 to 1.3.0 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2937](https://togithub.com/99designs/gqlgen/pull/2937) - Bump graphql-ws from 5.14.3 to 5.15.0 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2935](https://togithub.com/99designs/gqlgen/pull/2935) - Bump vite from 5.1.1 to 5.1.3 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2936](https://togithub.com/99designs/gqlgen/pull/2936) - Bump golang.org/x/tools from 0.17.0 to 0.18.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2940](https://togithub.com/99designs/gqlgen/pull/2940) - Optionally render entity requires populator function for advanced [@​requires](https://togithub.com/requires) use cases by [@​jesse-apollo](https://togithub.com/jesse-apollo) in [https://github.com/99designs/gqlgen/pull/2884](https://togithub.com/99designs/gqlgen/pull/2884) - Bump vite from 5.1.3 to 5.1.4 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2947](https://togithub.com/99designs/gqlgen/pull/2947) - Bump [@​graphql-codegen/introspection](https://togithub.com/graphql-codegen/introspection) from 4.0.2 to 4.0.3 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2944](https://togithub.com/99designs/gqlgen/pull/2944) - Bump [@​graphql-codegen/client-preset](https://togithub.com/graphql-codegen/client-preset) from 4.2.2 to 4.2.4 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2945](https://togithub.com/99designs/gqlgen/pull/2945) - Bump github.com/PuerkitoBio/goquery from 1.8.1 to 1.9.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2943](https://togithub.com/99designs/gqlgen/pull/2943) - Bump vitest from 1.3.0 to 1.3.1 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2946](https://togithub.com/99designs/gqlgen/pull/2946) - Bump github.com/stretchr/testify from 1.8.4 to 1.9.0 in /\_examples by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2955](https://togithub.com/99designs/gqlgen/pull/2955) - Bump github.com/PuerkitoBio/goquery from 1.9.0 to 1.9.1 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2954](https://togithub.com/99designs/gqlgen/pull/2954) - Bump github.com/stretchr/testify from 1.8.4 to 1.9.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2953](https://togithub.com/99designs/gqlgen/pull/2953) - Add option to omit resolver fields from models by [@​Desuuuu](https://togithub.com/Desuuuu) in [https://github.com/99designs/gqlgen/pull/2957](https://togithub.com/99designs/gqlgen/pull/2957) - fix(docs): convert an unnecessarily capitalized word to lowercase by [@​hxrxchang](https://togithub.com/hxrxchang) in [https://github.com/99designs/gqlgen/pull/2959](https://togithub.com/99designs/gqlgen/pull/2959) - Update explicit_requires to support models generated into same package by [@​ericbock](https://togithub.com/ericbock) in [https://github.com/99designs/gqlgen/pull/2965](https://togithub.com/99designs/gqlgen/pull/2965) - Add case for resolvers_always_return_pointers:false in explicit requires generation. by [@​jesse-apollo](https://togithub.com/jesse-apollo) in [https://github.com/99designs/gqlgen/pull/2966](https://togithub.com/99designs/gqlgen/pull/2966) - Bump google.golang.org/protobuf from 1.32.0 to 1.33.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2964](https://togithub.com/99designs/gqlgen/pull/2964) - Bump [@​apollo/client](https://togithub.com/apollo/client) from 3.9.5 to 3.9.6 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2962](https://togithub.com/99designs/gqlgen/pull/2962) - Bump vite from 5.1.4 to 5.1.5 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2961](https://togithub.com/99designs/gqlgen/pull/2961) - Bump typescript from 5.3.3 to 5.4.2 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2960](https://togithub.com/99designs/gqlgen/pull/2960) - Bump golang.org/x/tools from 0.18.0 to 0.19.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2963](https://togithub.com/99designs/gqlgen/pull/2963) #### New Contributors - [@​jesse-apollo](https://togithub.com/jesse-apollo) made their first contribution in [https://github.com/99designs/gqlgen/pull/2884](https://togithub.com/99designs/gqlgen/pull/2884) - [@​hxrxchang](https://togithub.com/hxrxchang) made their first contribution in [https://github.com/99designs/gqlgen/pull/2959](https://togithub.com/99designs/gqlgen/pull/2959) - [@​ericbock](https://togithub.com/ericbock) made their first contribution in [https://github.com/99designs/gqlgen/pull/2965](https://togithub.com/99designs/gqlgen/pull/2965) **Full Changelog**: 99designs/gqlgen@v0.17.44...v0.17.45 ### [`v0.17.44`](https://togithub.com/99designs/gqlgen/releases/tag/v0.17.44) [Compare Source](https://togithub.com/99designs/gqlgen/compare/v0.17.43...v0.17.44) #### What's Changed - Bump vite from 4.3.9 to 4.5.2 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2885](https://togithub.com/99designs/gqlgen/pull/2885) - Update federation plugin by [@​trevor-scheer](https://togithub.com/trevor-scheer) in [https://github.com/99designs/gqlgen/pull/2876](https://togithub.com/99designs/gqlgen/pull/2876) - Work with https://specs.apollo.dev/federation/v2.x by [@​StevenACoffman](https://togithub.com/StevenACoffman) in [https://github.com/99designs/gqlgen/pull/2891](https://togithub.com/99designs/gqlgen/pull/2891) - Update x/tools and add go v1.21,v1.22 in CI workflows by [@​ryicoh](https://togithub.com/ryicoh) in [https://github.com/99designs/gqlgen/pull/2894](https://togithub.com/99designs/gqlgen/pull/2894) - Bump actions/setup-go from 3 to 5 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2900](https://togithub.com/99designs/gqlgen/pull/2900) - Bump actions/checkout from 3 to 4 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2903](https://togithub.com/99designs/gqlgen/pull/2903) - Bump actions/setup-node from 3 to 4 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2910](https://togithub.com/99designs/gqlgen/pull/2910) - Bump nick-fields/retry from 2 to 3 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2907](https://togithub.com/99designs/gqlgen/pull/2907) - Bump golangci/golangci-lint-action from 3.5.0 to 3.7.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2896](https://togithub.com/99designs/gqlgen/pull/2896) - Bump github.com/stretchr/testify from 1.8.2 to 1.8.4 in /\_examples by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2897](https://togithub.com/99designs/gqlgen/pull/2897) - Bump github.com/gorilla/websocket from 1.5.0 to 1.5.1 in /\_examples by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2901](https://togithub.com/99designs/gqlgen/pull/2901) - Bump github.com/matryer/moq from 0.2.7 to 0.3.3 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2902](https://togithub.com/99designs/gqlgen/pull/2902) - Bump vitest from 0.32.0 to 1.2.2 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2919](https://togithub.com/99designs/gqlgen/pull/2919) - Bump styled-components from 5.3.11 to 6.1.8 in /\_examples/chat by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2905](https://togithub.com/99designs/gqlgen/pull/2905) - Bump graphql-sse from 2.1.4 to 2.5.2 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2913](https://togithub.com/99designs/gqlgen/pull/2913) - Bump typescript from 4.9.5 to 5.3.3 in /\_examples/chat by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2917](https://togithub.com/99designs/gqlgen/pull/2917) - Bump react-scripts from 2.1.8 to 5.0.1 in /\_examples/chat by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2914](https://togithub.com/99designs/gqlgen/pull/2914) - Bump github.com/urfave/cli/v2 from 2.25.5 to 2.27.1 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2912](https://togithub.com/99designs/gqlgen/pull/2912) - Bump subscriptions-transport-ws from 0.9.19 to 0.11.0 in /\_examples/chat by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2911](https://togithub.com/99designs/gqlgen/pull/2911) - Bump github.com/google/uuid from 1.3.0 to 1.6.0 in /\_examples by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2909](https://togithub.com/99designs/gqlgen/pull/2909) - Bump github.com/rs/cors from 1.9.0 to 1.10.1 in /\_examples by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2904](https://togithub.com/99designs/gqlgen/pull/2904) - Bump graphql from 14.7.0 to 16.8.1 in /\_examples/chat by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2899](https://togithub.com/99designs/gqlgen/pull/2899) - Bump jest from 25.5.4 to 29.7.0 in /\_examples/federation by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2920](https://togithub.com/99designs/gqlgen/pull/2920) - Bump typescript from 5.1.3 to 5.3.3 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2921](https://togithub.com/99designs/gqlgen/pull/2921) - Bump [@​graphql-codegen/schema-ast](https://togithub.com/graphql-codegen/schema-ast) from 4.0.0 to 4.0.2 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2918](https://togithub.com/99designs/gqlgen/pull/2918) - Bump github.com/mattn/go-isatty from 0.0.19 to 0.0.20 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2908](https://togithub.com/99designs/gqlgen/pull/2908) - Bump urql from 4.0.4 to 4.0.6 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2906](https://togithub.com/99designs/gqlgen/pull/2906) - Bump github.com/hashicorp/golang-lru/v2 from 2.0.3 to 2.0.7 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2915](https://togithub.com/99designs/gqlgen/pull/2915) - Bump node-fetch from 2.7.0 to 3.3.2 in /\_examples/federation by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2916](https://togithub.com/99designs/gqlgen/pull/2916) - graphql/uint: Fix unmarshalling of negative numbers by [@​mnPanic](https://togithub.com/mnPanic) in [https://github.com/99designs/gqlgen/pull/2922](https://togithub.com/99designs/gqlgen/pull/2922) - Bump golangci/golangci-lint-action from 3.7.0 to 4.0.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2928](https://togithub.com/99designs/gqlgen/pull/2928) - Bump github.com/google/uuid from 1.3.0 to 1.6.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2924](https://togithub.com/99designs/gqlgen/pull/2924) - Bump github.com/stretchr/testify from 1.8.2 to 1.8.4 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2925](https://togithub.com/99designs/gqlgen/pull/2925) - Bump google.golang.org/protobuf from 1.30.0 to 1.32.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2926](https://togithub.com/99designs/gqlgen/pull/2926) - Bump react from 16.14.0 to 18.2.0 in /\_examples/chat by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2929](https://togithub.com/99designs/gqlgen/pull/2929) - Bump vite from 4.5.2 to 5.1.1 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2931](https://togithub.com/99designs/gqlgen/pull/2931) - Bump [@​graphql-codegen/cli](https://togithub.com/graphql-codegen/cli) from 4.0.1 to 5.0.2 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2932](https://togithub.com/99designs/gqlgen/pull/2932) - Bump github.com/sosodev/duration from 1.1.0 to 1.2.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2927](https://togithub.com/99designs/gqlgen/pull/2927) - Bump react-dom from 16.14.0 to 18.2.0 in /\_examples/chat by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2930](https://togithub.com/99designs/gqlgen/pull/2930) #### New Contributors - [@​trevor-scheer](https://togithub.com/trevor-scheer) made their first contribution in [https://github.com/99designs/gqlgen/pull/2876](https://togithub.com/99designs/gqlgen/pull/2876) - [@​ryicoh](https://togithub.com/ryicoh) made their first contribution in [https://github.com/99designs/gqlgen/pull/2894](https://togithub.com/99designs/gqlgen/pull/2894) - [@​mnPanic](https://togithub.com/mnPanic) made their first contribution in [https://github.com/99designs/gqlgen/pull/2922](https://togithub.com/99designs/gqlgen/pull/2922) **Full Changelog**: 99designs/gqlgen@v0.17.43...v0.17.44 ### [`v0.17.43`](https://togithub.com/99designs/gqlgen/releases/tag/v0.17.43) [Compare Source](https://togithub.com/99designs/gqlgen/compare/v0.17.42...v0.17.43) #### What's Changed - Fix code generation for federated multi-key, multi-entity types by [@​bpeters-cmu](https://togithub.com/bpeters-cmu) in [https://github.com/99designs/gqlgen/pull/2877](https://togithub.com/99designs/gqlgen/pull/2877) - Add config option to omit root objects from models by [@​ghjm](https://togithub.com/ghjm) in [https://github.com/99designs/gqlgen/pull/2878](https://togithub.com/99designs/gqlgen/pull/2878) - add omitempty config to example gqlgen.yml by [@​PaulVasilenko](https://togithub.com/PaulVasilenko) in [https://github.com/99designs/gqlgen/pull/2880](https://togithub.com/99designs/gqlgen/pull/2880) - Update gqlparser to v2.5.11 by [@​StevenACoffman](https://togithub.com/StevenACoffman) in [https://github.com/99designs/gqlgen/pull/2882](https://togithub.com/99designs/gqlgen/pull/2882) #### New Contributors - [@​bpeters-cmu](https://togithub.com/bpeters-cmu) made their first contribution in [https://github.com/99designs/gqlgen/pull/2877](https://togithub.com/99designs/gqlgen/pull/2877) - [@​ghjm](https://togithub.com/ghjm) made their first contribution in [https://github.com/99designs/gqlgen/pull/2878](https://togithub.com/99designs/gqlgen/pull/2878) - [@​PaulVasilenko](https://togithub.com/PaulVasilenko) made their first contribution in [https://github.com/99designs/gqlgen/pull/2880](https://togithub.com/99designs/gqlgen/pull/2880) **Full Changelog**: 99designs/gqlgen@v0.17.42...v0.17.43 ### [`v0.17.42`](https://togithub.com/99designs/gqlgen/blob/HEAD/CHANGELOG.md#v01742---2023-12-29) [Compare Source](https://togithub.com/99designs/gqlgen/compare/v0.17.41...v0.17.42) - <a href="https://togithub.com/99designs/gqlgen/commit/7bf0c223aec642d086793698bc2a0d1a6fdb09b4"><tt>[`7bf0c22`](https://togithub.com/99designs/gqlgen/commit/7bf0c223)</tt></a> release v0.17.42 <dl><dd><details><summary><a href="https://togithub.com/99designs/gqlgen/commit/c811d47ec498bdd50591f163e7d23a7524e98280"><tt>c811d47e</tt></a> fix: avoid panic from tracing on bad request (<a href="https://togithub.com/99designs/gqlgen/pull/2871">#​2871</a>)</summary> This fixes a panic which arises from the tracing components when a request has some defect which results in an error when creating the operation context. The transports consistently handle this by calling `DispatchError(graphql.WithOperationContext(ctx, rc), err)` where `rc` is the OperationContext which was not correctly constructed. This seems dangerous, because middleware may assume that if there in an `OperationContext` in the `context.Context` than they are being invoked on a normal codepath and can assume their other interceptors have been invoked in the normal order. Also, using a value returned by a function which also returned a non-nil error is very unusual. However, I have no idea what the impact of changing that dangerous behavior in the transports would be, so I opted to make the tracing component more resilient instead. </details></dd></dl> - <a href="https://togithub.com/99designs/gqlgen/commit/13bb415268dda837690835e65e331746c8df892b"><tt>[`13bb415`](https://togithub.com/99designs/gqlgen/commit/13bb4152)</tt></a> fix for entity interfce code gen with related test (<a href="https://togithub.com/99designs/gqlgen/pull/2868">[#​2868](https://togithub.com/99designs/gqlgen/issues/2868)</a>) - <a href="https://togithub.com/99designs/gqlgen/commit/0354649c0309af6acfe089d12d103060d55a5805"><tt>[`0354649`](https://togithub.com/99designs/gqlgen/commit/0354649c)</tt></a> Remove archived dependency appdash (<a href="https://togithub.com/99designs/gqlgen/pull/2866">[#​2866](https://togithub.com/99designs/gqlgen/issues/2866)</a>) - <a href="https://togithub.com/99designs/gqlgen/commit/0d43599cdab22912d4ddd061c3b3ffd5d8da3845"><tt>[`0d43599`](https://togithub.com/99designs/gqlgen/commit/0d43599c)</tt></a> Update examples go.mod with appdash replacements (<a href="https://togithub.com/99designs/gqlgen/pull/2863">[#​2863](https://togithub.com/99designs/gqlgen/issues/2863)</a>) - <a href="https://togithub.com/99designs/gqlgen/commit/7dd971c871c0b0159ad26c9bf3095a8ba3780402"><tt>[`7dd971c`](https://togithub.com/99designs/gqlgen/commit/7dd971c8)</tt></a> Use defer wg.Done() in FieldSet Dispatch (<a href="https://togithub.com/99designs/gqlgen/pull/2861">[#​2861](https://togithub.com/99designs/gqlgen/issues/2861)</a>) <dl><dd><details><summary><a href="https://togithub.com/99designs/gqlgen/commit/24ea195cebea095035caf4d23af7f3d75fd0a041"><tt>24ea195c</tt></a> vikstrous/dataloadgen replaces recommended dataloader package in example docs (<a href="https://togithub.com/99designs/gqlgen/pull/2770">#​2770</a>)</summary> - update example for dataloadgen - improved example with link to example repo - undo unnecessary changes - fix wrong signature - fix creation of loader - Update docs/content/reference/dataloaders.md </details></dd></dl> <dl><dd><details><summary><a href="https://togithub.com/99designs/gqlgen/commit/42f6e39d48e3a301bf39cd4e8fd180250bc25f2c"><tt>42f6e39d</tt></a> Allow fields that return root level definitions (<a href="https://togithub.com/99designs/gqlgen/pull/2858">#​2858</a>)</summary> - generate structs for root level definitions to support fields that return Query, Mutation or Subscription - removed unnecessary comment - re-ran go generate *** </details></dd></dl> - <a href="https://togithub.com/99designs/gqlgen/commit/682a58dd6af5fda53509fbf4cfa45d23b5bb1c86"><tt>[`682a58d`](https://togithub.com/99designs/gqlgen/commit/682a58dd)</tt></a> Add go generate for examples so contributors never forget (<a href="https://togithub.com/99designs/gqlgen/pull/2859">[#​2859](https://togithub.com/99designs/gqlgen/issues/2859)</a>) <dl><dd><details><summary><a href="https://togithub.com/99designs/gqlgen/commit/e080a96de178520fcfaf5a8d68836981ec4df9a9"><tt>e080a96d</tt></a> Modify to prevent unreachable code from occurring (<a href="https://togithub.com/99designs/gqlgen/pull/2846">#​2846</a>)</summary> - fix: 型の数でソートする処理を追加 - 戻し - fix: case文の最初にスーパークラスが来ないようにする - testdata追加 - fix: Added sorting by number of types. - fix: Prevent superclass from appearing at the beginning of case statement </details></dd></dl> - <a href="https://togithub.com/99designs/gqlgen/commit/68744ad2a1e9d5869ab6a00b49814c6ae9583186"><tt>[`68744ad`](https://togithub.com/99designs/gqlgen/commit/68744ad2)</tt></a> Bump changelog - <a href="https://togithub.com/99designs/gqlgen/commit/e4cf21d24518deb99af6d4c0ea86de11d6889349"><tt>[`e4cf21d`](https://togithub.com/99designs/gqlgen/commit/e4cf21d2)</tt></a> v0.17.41 postrelease bump <!-- end of Commits --> <!-- end of Else --> <!-- end of If NoteGroups --> ### [`v0.17.41`](https://togithub.com/99designs/gqlgen/blob/HEAD/CHANGELOG.md#v01741---2023-12-03) [Compare Source](https://togithub.com/99designs/gqlgen/compare/v0.17.40...v0.17.41) - <a href="https://togithub.com/99designs/gqlgen/commit/fe60938c55308b1cd5562556cdb976771cfcc6cc"><tt>[`fe60938`](https://togithub.com/99designs/gqlgen/commit/fe60938c)</tt></a> release v0.17.41 <dl><dd><details><summary><a href="https://togithub.com/99designs/gqlgen/commit/5e98a16a3a5a5678f1b6481275d81f52f9462f90"><tt>5e98a16a</tt></a> fix fieldset.New bug when prefix slice has len < cap (<a href="https://togithub.com/99designs/gqlgen/pull/2851">#​2851</a>)</summary> - fix fieldset.New bug when prefix slice has len < cap - ignore gocritic warning </details></dd></dl> <dl><dd><details><summary><a href="https://togithub.com/99designs/gqlgen/commit/bd9657f3e50b7b9642c05039a5364ce2262faaf4"><tt>bd9657f3</tt></a> Improve ResolverImplementer.Implment (<a href="https://togithub.com/99designs/gqlgen/pull/2850">#​2850</a>)</summary> - improve resolver implement render - add error when multiple implementors - add initial test </details></dd></dl> <dl><dd><details><summary><a href="https://togithub.com/99designs/gqlgen/commit/cb3c1c890e5a255776df9577c80b0c15218cf254"><tt>cb3c1c89</tt></a> Updated apollo sandbox (<a href="https://togithub.com/99designs/gqlgen/pull/2849">#​2849</a>)</summary> Added all supported options to new window.EmbeddedSandbox object </details></dd></dl> <dl><dd><details><summary><a href="https://togithub.com/99designs/gqlgen/commit/eb5cea7206767bda3582040fd9e4c98174aaa6b1"><tt>eb5cea72</tt></a> Small template fix to save space in the generated file (<a href="https://togithub.com/99designs/gqlgen/pull/2841">#​2841</a>)</summary> - Small template fix to save space in the generated file - Re-generate *** </details></dd></dl> - <a href="https://togithub.com/99designs/gqlgen/commit/c0ca5091a10417c34192da4d3c064a0fed2a7fdb"><tt>[`c0ca509`](https://togithub.com/99designs/gqlgen/commit/c0ca5091)</tt></a> Omittable can now be serialized as json (<a href="https://togithub.com/99designs/gqlgen/pull/2839">[#​2839](https://togithub.com/99designs/gqlgen/issues/2839)</a>) - <a href="https://togithub.com/99designs/gqlgen/commit/dcb7619111642cc82a21b8e80ce1300213af1368"><tt>[`dcb7619`](https://togithub.com/99designs/gqlgen/commit/dcb76191)</tt></a> fix: sample program indentation (<a href="https://togithub.com/99designs/gqlgen/pull/2840">[#​2840](https://togithub.com/99designs/gqlgen/issues/2840)</a>) <dl><dd><details><summary><a href="https://togithub.com/99designs/gqlgen/commit/132ec1ce579e9ce3dc772afebf9703c1403d588e"><tt>132ec1ce</tt></a> Updated GraphiQL 3.0.1 => 3.0.6 (<a href="https://togithub.com/99designs/gqlgen/pull/2837">#​2837</a>)</summary> - Updated GraphiQL 3.0.1 => 3.0.6 - Added unit tests to cover integrity of playgrounds - Updated vulnerable dependency - Close response body </details></dd></dl> - <a href="https://togithub.com/99designs/gqlgen/commit/917407005eb4198aa43875984fb77caeaa7fca36"><tt>[`9174070`](https://togithub.com/99designs/gqlgen/commit/91740700)</tt></a> v0.17.40 postrelease bump <!-- end of Commits --> <!-- end of Else --> <!-- end of If NoteGroups --> ### [`v0.17.40`](https://togithub.com/99designs/gqlgen/releases/tag/v0.17.40) [Compare Source](https://togithub.com/99designs/gqlgen/compare/v0.17.39...v0.17.40) ##### What's Changed - resolver: fix case-insensitive file name collision by [@​erankor](https://togithub.com/erankor) in [https://github.com/99designs/gqlgen/pull/2829](https://togithub.com/99designs/gqlgen/pull/2829) - Bump [@​babel/traverse](https://togithub.com/babel/traverse) from 7.22.5 to 7.23.2 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2831](https://togithub.com/99designs/gqlgen/pull/2831) - Map based input types fields are now coerced to the right type by [@​endSly](https://togithub.com/endSly) in [https://github.com/99designs/gqlgen/pull/2830](https://togithub.com/99designs/gqlgen/pull/2830) **Full Changelog**: 99designs/gqlgen@v0.17.39...v0.17.40 ### [`v0.17.39`](https://togithub.com/99designs/gqlgen/releases/tag/v0.17.39) [Compare Source](https://togithub.com/99designs/gqlgen/compare/v0.17.38...v0.17.39) #### What's Changed - Breaking API: Allow WebsocketInitFunc to add payload to Ack ([#​4](https://togithub.com/99designs/gqlgen/issues/4)) by [@​telemenar](https://togithub.com/telemenar) in [https://github.com/99designs/gqlgen/pull/2791](https://togithub.com/99designs/gqlgen/pull/2791) - add close flag into wsConnection to avoid duplicate calls of CloseFunc by [@​vlad-tokarev](https://togithub.com/vlad-tokarev) in [https://github.com/99designs/gqlgen/pull/2803](https://togithub.com/99designs/gqlgen/pull/2803) - fix: CodeGen for omit_slice_element_pointers and GetMany Entity Resolvers by [@​parkerroan](https://togithub.com/parkerroan) in [https://github.com/99designs/gqlgen/pull/2802](https://togithub.com/99designs/gqlgen/pull/2802) - feat: update getting-started CreateTodo mutationResolver by [@​gitxiongpan](https://togithub.com/gitxiongpan) in [https://github.com/99designs/gqlgen/pull/2810](https://togithub.com/99designs/gqlgen/pull/2810) - Feature: Support Apollo Federation Auth Directives by [@​parkerroan](https://togithub.com/parkerroan) in [https://github.com/99designs/gqlgen/pull/2809](https://togithub.com/99designs/gqlgen/pull/2809) - Consider go type name when autobinding by [@​dany74q](https://togithub.com/dany74q) in [https://github.com/99designs/gqlgen/pull/2812](https://togithub.com/99designs/gqlgen/pull/2812) - Update generated files that change when building by [@​telemenar](https://togithub.com/telemenar) in [https://github.com/99designs/gqlgen/pull/2813](https://togithub.com/99designs/gqlgen/pull/2813) - Add a pong only keep alive for the new protocol by [@​telemenar](https://togithub.com/telemenar) in [https://github.com/99designs/gqlgen/pull/2814](https://togithub.com/99designs/gqlgen/pull/2814) - Store parsed Schema on ExecutableSchema rather than use global variable by [@​gitxiongpan](https://togithub.com/gitxiongpan) in [https://github.com/99designs/gqlgen/pull/2811](https://togithub.com/99designs/gqlgen/pull/2811) - Add ability to not fail when pong is not received. by [@​telemenar](https://togithub.com/telemenar) in [https://github.com/99designs/gqlgen/pull/2815](https://togithub.com/99designs/gqlgen/pull/2815) - Adding duration scalar conforming to ISO8601 standard by [@​rwrz](https://togithub.com/rwrz) in [https://github.com/99designs/gqlgen/pull/2800](https://togithub.com/99designs/gqlgen/pull/2800) - Bump postcss from 8.4.24 to 8.4.31 in /integration by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/99designs/gqlgen/pull/2819](https://togithub.com/99designs/gqlgen/pull/2819) - Add [@​interfaceObject](https://togithub.com/interfaceObject) and [@​composeDirective](https://togithub.com/composeDirective) at Federation 2 directive lists. by [@​wangmir](https://togithub.com/wangmir) in [https://github.com/99designs/gqlgen/pull/2821](https://togithub.com/99designs/gqlgen/pull/2821) #### New Contributors - [@​vlad-tokarev](https://togithub.com/vlad-tokarev) made their first contribution in [https://github.com/99designs/gqlgen/pull/2803](https://togithub.com/99designs/gqlgen/pull/2803) - [@​parkerroan](https://togithub.com/parkerroan) made their first contribution in [https://github.com/99designs/gqlgen/pull/2802](https://togithub.com/99designs/gqlgen/pull/2802) - [@​dany74q](https://togithub.com/dany74q) made their first contribution in [https://github.com/99designs/gqlgen/pull/2812](https://togithub.com/99designs/gqlgen/pull/2812) - [@​rwrz](https://togithub.com/rwrz) made their first contribution in [https://github.com/99designs/gqlgen/pull/2800](https://togithub.com/99designs/gqlgen/pull/2800) - [@​wangmir](https://togithub.com/wangmir) made their first contribution in [https://github.com/99designs/gqlgen/pull/2821](https://togithub.com/99designs/gqlgen/pull/2821) **Full Changelog**: 99designs/gqlgen@v0.17.38...v0.17.39 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/infratographer/x). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4wLjMiLCJ1cGRhdGVkSW5WZXIiOiIzNy4yMzAuMCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I also changed how the read deadline set works a little, the reason for this is that the protocol allows for pong to be sent without a ping.
So setting a read deadline on receiving pong isn't great. Instead we should always set the read deadline on sending ping. Though to do this we need to know whether we have received a pong or not. Because if we set the read deadline when the previous ping still hasn't received the pong. Then it will never hit the deadline.
I have: