Douglas Wilson

Hello, and very sorry about that! I tried to make the regular expression match the grammar as defined in RFC 5987. Looking at the spec, I don't believe either of...

Ah, yes, it looks like this is a new change in the spec, as that requirement was not part of RFC 7232, AFAICT.

Oh, you're right. That's why I said as far as I can tell. I guess I assumed it would be in the same section in both RFC, doh.

Hi @marco2216 you can, though sorry I didn't post yet, just before you go through too much trouble, I did already start on the `develop` branch a couple hours ago....

Hi @bigbigDreamer apologies for that! It just got lost in the sea of notifications for me and then fell off more as I got some unrelated security reports I had...

I haven't looked at `--frozen-intrinsics` since Node.js 18 ar whichit was an experimental feature that Node.js itself said would break code and I asked about this and they confirmed there...

Ok, thanks. When I get home tonight I will file another bug report on Node.js to see what the the solution is or to re make them aware it is...

Typically the readme code is supported to work on the same versions of Node.js that the module itself works on, though.

Unless upsteam Apache actually changed the mapping, your change will be overwritten because you changed a generated file. You should contact Apache project for changes to Apache mappings.

Ya, there isn't a huge reason to update the built in certs here, as the new module splits them out. The separate module is great too bc then updating certs...