Closed npm1 closed 2 years ago
PTAL, addressed all of your comments. The pending thing is the referrer being handwavy (in the case we do want to send it), while I figure out how to set it properly.
Ready for more review, PTAL.
Looks generally good to me, but still feels like the parameters aren't quite right.
On a separate note, any chance we can get @domfarolino to take a pass at this too?
Can you look again but looking at the latest version??
Added Sec-
header changes
LGTM
I'll merge, and if there is anything pending we can patch separately.
Relevant issues:
This PR adds the fetch calls to the other FedCM fetches.
Preview | Diff