VividCortex/siesta

Don't use MustCompile() for dynamically built regexes

Closed this issue · 2 comments

gkristic commented

This is a follow up on #15. The use of MustCompile() for a regular expression can easily crash an application. I suggest using Compile() instead and handling the error appropriately.

Preetam commented

There's work to remove regular expressions completely: #14

gkristic commented

Okay. If that's ongoing work then this issue makes no sense. Closing!