fix(plugin-server): trackedFetch lookups #25398
Merged
+32
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Problem
The node-js fetch function didn't validate DNS records for each hop of a redirect.
Production uses a rust based system instead which does not appear to be affected.
Changes
Provide a custom DNS lookup function to node-fetch.
How did you test this code?
I made a fetch request locally to a URL that redirects, and saw the lookup function called twice.