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(deps): update all non-major dependencies #6803

Merged
merged 2 commits into from
Dec 26, 2024
Merged

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Dec 26, 2024

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
@graphql-tools/executor-graphql-ws (source) 1.3.6 -> 1.3.7 age adoption passing confidence
@graphql-tools/executor-http (source) 1.2.2 -> 1.2.3 age adoption passing confidence

Release Notes

graphql-hive/gateway (@​graphql-tools/executor-graphql-ws)

v1.3.7

Compare Source

Patch Changes
  • #​381 55eb1b4 Thanks @​ardatan! - dependencies updates:

  • #​381 55eb1b4 Thanks @​ardatan! - This is a bugfix with some internal changes, no user action is needed. This bugfix and improvement is done to improve the stability of some components of the gateway;

    Like HMAC Upstream Signature plugin, different components of the gateway were using different ways of serializing the execution request.
    Some of them were ignoring variables if it is empty, some of not, this was causing the signature generation to be different for the same query.
    For example, it was working as expected in Proxy mode, but not working as expected in Federation Gateway mode.

    With this change, now we have a shared helper to serialize the upstream execution request with a memoized print function for query AST etc to have a consistent serialization so consistent signature generation for HMAC.

    For example instead of using print, you should use defaultPrintFn that memoizes print operation and also used the string version of it parsed before by Envelop/Yoga.

    -import { print } from 'graphql';
    -const query = print(parsedQuery);
    +import { defaultPrintFn } from '@​graphql-tools/executor-common';
    +const query = defaultPrintFn(parsedQuery);

    Or instead of creating objects from ExecutionRequest, use serializeExecutionRequest helper.

    -const serializedRequest = {
    -  query: print(executionRequest.document),
    -  variables: executionRequest.variables,
    -  operationName: executionRequest.operationName,
    -  extensions: executionRequest.extensions,
    -};
    +import { serializeExecutionRequest } from '@​graphql-tools/executor-common';
    +const serializedRequest = serializeExecutionRequest(executionRequest);
  • Updated dependencies [55eb1b4]:

graphql-hive/gateway (@​graphql-tools/executor-http)

v1.2.3

Compare Source

Patch Changes
  • #​381 55eb1b4 Thanks @​ardatan! - dependencies updates:

  • #​381 55eb1b4 Thanks @​ardatan! - This is a bugfix with some internal changes, no user action is needed. This bugfix and improvement is done to improve the stability of some components of the gateway;

    Like HMAC Upstream Signature plugin, different components of the gateway were using different ways of serializing the execution request.
    Some of them were ignoring variables if it is empty, some of not, this was causing the signature generation to be different for the same query.
    For example, it was working as expected in Proxy mode, but not working as expected in Federation Gateway mode.

    With this change, now we have a shared helper to serialize the upstream execution request with a memoized print function for query AST etc to have a consistent serialization so consistent signature generation for HMAC.

    For example instead of using print, you should use defaultPrintFn that memoizes print operation and also used the string version of it parsed before by Envelop/Yoga.

    -import { print } from 'graphql';
    -const query = print(parsedQuery);
    +import { defaultPrintFn } from '@​graphql-tools/executor-common';
    +const query = defaultPrintFn(parsedQuery);

    Or instead of creating objects from ExecutionRequest, use serializeExecutionRequest helper.

    -const serializedRequest = {
    -  query: print(executionRequest.document),
    -  variables: executionRequest.variables,
    -  operationName: executionRequest.operationName,
    -  extensions: executionRequest.extensions,
    -};
    +import { serializeExecutionRequest } from '@​graphql-tools/executor-common';
    +const serializedRequest = serializeExecutionRequest(executionRequest);
  • Updated dependencies [55eb1b4]:


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.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate renovate bot added the dependencies Pull requests that update a dependency file label Dec 26, 2024
Copy link

changeset-bot bot commented Dec 26, 2024

⚠️ No Changeset found

Latest commit: 98d477f

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.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

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

Copy link
Contributor

github-actions bot commented Dec 26, 2024

💻 Website Preview

The latest changes are available as preview in: https://69f5d8a7.graphql-tools.pages.dev

@ardatan ardatan merged commit 0625678 into master Dec 26, 2024
13 of 14 checks passed
@ardatan ardatan deleted the renovate/all-minor-patch branch December 26, 2024 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant