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.

Sign up for GitHub

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

Jump to bottom

Getting errors with latest apollo server and apollo datasource versions #37

Closed
mmhaqq opened this issue Jun 3, 2023 · 2 comments
Closed

Getting errors with latest apollo server and apollo datasource versions #37

mmhaqq opened this issue Jun 3, 2023 · 2 comments

Comments

@mmhaqq
Copy link

mmhaqq commented Jun 3, 2023 •
edited

@apollo/server

version: 4.7.1
example: graphql-nodejs-cloudflare-workers
error: Uncaught Error: Some functionality, such as asynchronous I/O, timeouts, and generating random values, can only be performed while handling a request

works fine till v4.7.0.

apollo-datasource-rest

version: 6.0.0
error: header.raw is not a function when used headers.
similar report: kimyvgy/worker-apollo-server-template#36

working with v4.3.2
@trevor-scheer trevor-scheer mentioned this issue Jun 7, 2023
Merged
trevor-scheer added a commit to apollographql/apollo-server that referenced this issue Jun 7, 2023
@trevor-scheer
Allow landing page to be configured with a precomputed nonce (fix for…
75b668d 
… CF workers) (#7601)

#7539 introduced a `v4()` call from the `uuid` package which broke CF
workers
(apollo-server-integrations/apollo-server-integration-cloudflare-workers#37).
This change allows users to configure / precompute the `nonce` in
advance to avoid making crypto-y calls on startup (which CF workers
throws errors about).
@trevor-scheer trevor-scheer mentioned this issue Jun 7, 2023
Merged
@trevor-scheer
Copy link
Member

Thanks for the report and examples @mmhaqq, I was able to reproduce and fix both of those issues. The latest patches should have those resolved now.

@MejanH
Copy link

MejanH commented Jun 9, 2023

Thanks. was facing an error. but forgot the check the release page. it's working fine. I have one question. the PRECOMPUTED_NONCE variable, i mean the nonce value we are adding, should we change this from time to time? or is it ok to have a generated uuid value forever?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@trevor-scheer @MejanH @mmhaqq