Error trying to use in a Cloudflare Worker

[jaymakes11]

I'm seeing the following error while trying to use the driver in a Cloudflare Worker:

ReferenceError: XMLHttpRequest is not defined

This is making it impossible to use Fauna from Cloudflare Workers, unless there's some workaround that I haven't yet considered.

@aequasi Sounds like you're using Workers, are you seeing this issue?

Note: Originally opened this issue on the closed #182 but then realized it's likely best to create a new issue for it.

[BrunoQuaresma]

Hi @lolcoolkat thanks for the fix.

1. We used cross-fetch because some environments was having compatibility issues. So we need to find another solution instead of just remove it.

2. I have to check what it is doing.

[BrunoQuaresma]

@lolcoolkat NodeJS does not support fetch() natively so we need to use some polyfill like cross-fetch to make driver works on both environments (browser and nodejs). Unfortunately, we didn't find out a polyfill that was not using XMLHttpRequest however we are brainstorming to see what we can do about this issue so please, let us know if you have any ideas on your mind.

[eigilsagafos]

@BrunoQuaresma We use Axios in the browser and on our node backend. Have you considered using that?

[BrunoQuaresma]

@eigilsagafos I think axios uses XMLHttpRequests right?

[eigilsagafos]

Yeah... you are right. Was just looking at that.

[eigilsagafos]

I know some packages build for different environments and allow you to require like require("fauna/browser") and require("fauna/node"). Maybe you could even provide a barebones version where you could provide your own http adapter?

[BrunoQuaresma]

Yeah, we are thinking on this solution as well but it adds some extra effort to define a common API interface that should work for these environments. Sounds strange to me a serverless environment does not support core NodeJS stuff. 🤔

[BrunoQuaresma]

Yeah Cloudflare Workers run ontop of V8 and without all the NodeJS overhead.. that's why they are super quick and performant!

Cool, that makes sense.

Doesn't have to be exactly like that, but hopefully you get the idea :D And that config would allow you to specify whether to use cross-fetch or not?

I think it is a nice suggestion. We will evaluate that. Thanks for the contribution @lolcoolkat

[Ayplow]

Is there any reason cross-fetch doesn't just use the global fetch when it exists?

[BrunoQuaresma]

@lolcoolkat yes it is! We just need to add some tests and an extra environment in our pipeline to check things on V8 environments. Thanks for the suggestion.

[BrunoQuaresma]

Fixes on https://www.npmjs.com/package/faunadb/v/2.11.2-beta. We are releasing this soon on the main version.

[fireridlle]

just a small note. Cloudflare doesn't implement AbortController which we need to terminate requests as well as streams. so it better to pass customized fetch. for example:

            const client = new faunadb.Client({
                'secret': 'your secret',
                fetch: (url, params) => {
                    const signal = params.signal
                    delete params.signal
                    const abortPromise = new Promise(resolve => {
                        if(signal) {
                            signal.onabort = resolve
                        }
                    })
                    return Promise.race([abortPromise, fetch(url, params)])
                }
            })

[Mhmdrza]

just a small note. Cloudflare doesn't implement AbortController which we need to terminate requests as well as streams. so it better to pass customized fetch. for example:

            const client = new faunadb.Client({
                'secret': 'your secret',
                fetch: (url, params) => {
                    const signal = params.signal
                    delete params.signal
                    const abortPromise = new Promise(resolve => {
                        if(signal) {
                            signal.onabort = resolve
                        }
                    })
                    return Promise.race([abortPromise, fetch(url, params)])
                }
            })

this one actually works :O) previous solution fetch.bind(globalThis) Isn't work with fauna v4.x.x