destr icon indicating copy to clipboard operation
destr copied to clipboard

perf: improve JSON signature regex for performance and clarity

Open mancuoj opened this issue 1 year ago • 1 comments

Changed capturing groups to non-capturing groups in JsonSigRx regex to slightly improve performance, as captured groups were not being used.

mancuoj avatar Aug 10 '24 17:08 mancuoj

Thanks for PR but can you please also share a benchmark diff report against both Node.js and Bun? 🙏🏼

(since this is a security area, i rather to minimize changes and only make it if it really makes a change)

pi0 avatar Aug 12 '24 12:08 pi0