fix(ui): bulk upload issues

[lukebennett]

What?

This PR contains a couple of fixes to the bulk upload process:

• Credentials not being passed when fetching, leading to auth issues
• Provide a fallback to crypto.randomUUID which is only available when using HTTPS or localhost

Why?

I use separate admin and API URLs and work off a remote dev server using custom hostnames. These issues may not impact the happy path of using localhost, but are dealbreakers in this environment.

Fixes

These are issues I found myself, I fixed them rather than raising issues for somebody else to pick up, but I can create issues to link to if required.

[JarrodMFlesch]

I'm not sure what we do everywhere else, but this actually will evaluate to undefined/api/route if you don't define a serverURL. Can you see what we do elsewhere?

[lukebennett]

I'm not sure what we do everywhere else, but this actually will evaluate to undefined/api/route if you don't define a serverURL. Can you see what we do elsewhere?

It's already depended on elsewhere in the same file e.g.

payload/packages/ui/src/elements/BulkUpload/FormsManager/index.tsx

Line 138 in 155dceb

const res = await fetch(`${serverURL}${api}${docAccessURL}?${qs.stringify(params)}`, {

and

payload/packages/ui/src/elements/BulkUpload/FormsManager/index.tsx

Line 149 in 155dceb

`${serverURL}${api}${docAccessURL}?${qs.stringify(params)}`,

so there's no new dependency introduced here.

Besides, it should never be undefined as it's given a default value of a blank string here:

payload/packages/payload/src/config/defaults.ts

Line 152 in 9c8f320

config.serverURL = config.serverURL ?? ''

[lukebennett]

The test failure is down to Docker Hub rate limiting, don't think there's anything I can really do about it

[lukebennett]

I've found another of these invalid fetch calls (that don't work with a custom API URL) in plugin-search. It's nothing to do with the bulk upoad but the same fix. Would you like me to add another commit here or open a new PR for it?

[JarrodMFlesch]

I've found another of these invalid fetch calls (that don't work with a custom API URL) in plugin-search. It's nothing to do with the bulk upoad but the same fix. Would you like me to add another commit here or open a new PR for it?

Another PR would be great.