OpenAPI handler: Passing API Key via query parameters + generate sdk issues

[coradinibr]

Is there a way to pass API Key via query parameters without having to create a resolver myself?
Here is a sandbox as example: https://codesandbox.io/s/angry-goldwasser-qsen0
If you run the example graphql query from above, you'll get a 401 (unauthorized) because qs parameters are not being added to the REST request.

In the same sandbox, if you try to generate the sdk by running yarn generate-sdk ./out.ts you get the error bellow.
I didn't investigate the issue in detail, but could be related to something similar to this post.

TypeError: Cannot read property 'length' of undefined
    at hasConditionalDirectives (/sandbox/node_modules/@graphql-codegen/visitor-plugin-common/index.cjs.js:479:20)
    at transformed._processor.transformPrimitiveFields.Array.from.map.field (/sandbox/node_modules/@graphql-codegen/visitor-plugin-common/index.cjs.js:2597:32)
    at Array.map (<anonymous>)
    at SelectionSetToObject.buildSelectionSetString (/sandbox/node_modules/@graphql-codegen/visitor-plugin-common/index.cjs.js:2596:112)
    at getPossibleTypes.reduce (/sandbox/node_modules/@graphql-codegen/visitor-plugin-common/index.cjs.js:2527:41)
    at Array.reduce (<anonymous>)
    at SelectionSetToObject._buildGroupedSelections (/sandbox/node_modules/@graphql-codegen/visitor-plugin-common/index.cjs.js:2517:80)
    at SelectionSetToObject.transformSelectionSet (/sandbox/node_modules/@graphql-codegen/visitor-plugin-common/index.cjs.js:2637:30)
    at SelectionSetToObject.buildSelectionSetString (/sandbox/node_modules/@graphql-codegen/visitor-plugin-common/index.cjs.js:2590:89)
    at getPossibleTypes.reduce (/sandbox/node_modules/@graphql-codegen/visitor-plugin-common/index.cjs.js:2527:41)

[ardatan]

@coradinibr Generating SDK without operations is still experimental so does the error remain when you provide your own operations for SDK(this would be more consistent for production usages)?

[coradinibr]

@ardatan, in the sandbox I provided, the Open Weather OpenAPI file has a get(200,404) operation defined (not custom).
As extra validation to your point, I created a mocked json schema and added a custom operation as test, then tried to generate the sdk again. Same error! Easy to reproduce by forking the example provided in graphql-mesh page here + add your custom operation, then try run generate-sdk.

Providing own operations for production makes total sense, but I can envision use cases where respecting the OpenAPI schema as defined by the downstream service won't be unsafe by any means.

[IMPORTANT]
I think this github issue should be split in 2 independent issues, but I guess would be better to have someone more knowledgeable to reproduce the sdk issue before creating a new issue. Thanks!

[ardatan]

Is this issue still remaining?
We recently released a new version of OpenAPI handler including a lot of new features and improvements.
I think you can pass additional query parameters with queryParams or headers with operationHeaders.
See the migration guide and the docs;
https://www.graphql-mesh.com/docs/migration/openapi-0.31-0.32

Closing this issue for now. We can reopen it later if the issue still happens.

[rmccorkl]

Can anyone provide an example of passing an API_KEY using the .meshrc.yaml file using either the queryParams or operationHeaders, nothing I've tried to date works and I end up with

"Failed to generate the schema GraphQLError: Enum "HTTPMethod" cannot represent value: "SERVERS".

In the previous version which had the header directive it was straightforward and simple, but I can't use the above directive to pass a simple API key...

I'm using node.js

[ardatan]

@rmccorkl Please create a new issue with a reproduction on CodeSandbox or StackBlitz.