Conversation

Notices

1. Embed this notice
   Aral Balkan (aral@mastodon.ar.al)'s status on Wednesday, 05-Jul-2023 03:58:34 JST Aral Balkan

   FML, something that’s changed between Node 12.16.2 and 18.16.0 has made the OCSP stapling library I’m using¹ 1000× slower (~4ms vs ~4s per request).

   At least I know where the issue is… Tomorrow, I dig into the library to try and narrow it down further.

   (The network graph in Firefox was useful as it showed that the problem was with the TLS Setup. And 0x² and its flame graphs helped me narrow it down further.)

   ¹ https://github.com/indutny/ocsp#ocsp-apis-for-nodejs-
   ² https://github.com/davidmarkclements/0x#readme

   #Kitten #TLS #OCSP #NodeJS #dev

   • Embed this notice
     Aral Balkan (aral@mastodon.ar.al)'s status on Wednesday, 05-Jul-2023 18:45:12 JST Aral Balkan

     in reply to

     • Fedor Indutny

     Right, well, whatever the regression is, it doesn’t exist in Node 20.3.1.

     To whit, the time it takes to run my SNICallback:

     Node 12.16.2: ~10ms
     Node 18.16.1: ~4,000ms
     Node 20.3.1: ~10ms

     So there’s been a *massive* regression in performance in something in Node 18 (LTS) – buffers/strings? – that appears to be fixed in Node 20 (but not backported) that’s affecting the ocsp module.

     CC @indutny – if you get a chance, could you possibly let the right folks at Node core know about this? Thanks :)

   • Embed this notice
     Aral Balkan (aral@mastodon.ar.al)'s status on Wednesday, 05-Jul-2023 18:49:10 JST Aral Balkan

     in reply to

     • Fedor Indutny

     @indutny PS. Profiling the ocsp.request.generate() function between versions should reproduce the performance regresssion in Node 18.

   • Embed this notice
     Aral Balkan (aral@mastodon.ar.al)'s status on Wednesday, 05-Jul-2023 18:59:20 JST Aral Balkan

     in reply to

     • Eric the Cerise

     @ErictheCerise Hey Eric, I’ll be updating Kitten to use Node 20.3.1 this week. You won’t notice any performance difference in development, this only impacts deployment.

     PS. Kitten uses its own Node runtime that it downloads during installation so the version of Node you have installed as your system default is irrelevant (so you don’t have to change that) :)

   • Embed this notice
     Eric the Cerise (ericthecerise@kolektiva.social)'s status on Wednesday, 05-Jul-2023 18:59:21 JST Eric the Cerise

     in reply to

     • Fedor Indutny

     @aral

     Just grabbed a copy of kitten 2 days ago, running on—you guessed it—Node 18 LTS.

     @indutny