URL.canParse fails when stressed #48816

Activity

1. joacub commented on Jul 17, 2023

   

   joacub

   on Jul 17, 2023

   Last edited by joacub

   Author

   More actions

   another test in my end doing a 100ms wait wil not have the same beharviour in that case return true always:

   import { setTimeout } from 'timers/promises';
   while (true) {
     await setTimeout(100);
     const test = URL.canParse('/ ', 'http://n');
     console.log(test);
   }

   this will also fail at some point so maybe the 100 ms will aso fail just didnt get the point were it will fail

   import { setTimeout } from 'timers/promises';
   while (true) {
     await setTimeout(1);
     const test = URL.canParse('/ ', 'http://n');
     console.log(test);
   }
2. joacub commented on Jul 17, 2023

   

   joacub

   on Jul 17, 2023

   Author

   More actions

   @anonrig i think you are following this changes so you may have all the details about this
3. anonrig

   added

   whatwg-urlIssues and PRs related to the WHATWG URL implementation.Issues and PRs related to the WHATWG URL implementation.

   confirmed-bugIssues and PRs for confirmed bugs.Issues and PRs for confirmed bugs.

   on Jul 17, 2023
4. anonrig commented on Jul 17, 2023

   

   anonrig

   on Jul 17, 2023

   Member

   More actions

   Thanks for the bug report. I confirm this is indeed a really awkward bug. cc @nodejs/url
5. anonrig commented on Jul 17, 2023

   

   anonrig

   on Jul 17, 2023

   Member

   More actions

   This is probably due to v8 Fast API. When you call it multiple times, it triggers v8 fast api.
6. anonrig commented on Jul 17, 2023

   

   anonrig

   on Jul 17, 2023

   Member

   More actions

   @KhafraDev disabling/commenting out V8 Fast API fixes the bug
7. anonrig commented on Jul 17, 2023

   

   anonrig

   on Jul 17, 2023

   Member

   More actions

   @joacub I opened a PR to fix this: #48817
8. joacub commented on Jul 17, 2023

   

   joacub

   on Jul 17, 2023

   Author

   More actions

   @joacub I opened a PR to fix this: #48817

   Thank you so much sir, we are facing many many issues due to this and our websites goes down very often, thank you so much for your work
9. kodiakhq

   added a commit that references this issue on Jul 18, 2023

   Revert "perf: improve URL validation performance" (#52818)

   ...

   33385aa
10. anonrig

    closed this as completedon Aug 1, 2023