yggdrasil-go icon indicating copy to clipboard operation
yggdrasil-go copied to clipboard

Panic when MulticastInterfaces/Regex has errors.

Open shibe2 opened this issue 1 year ago • 2 comments

Current behavior

https://github.com/yggdrasil-network/yggdrasil-go/blob/49c424ef213589d63215f10dc1958b2e93fb1e6c/cmd/yggdrasil/main.go#L242

MustCompile is like Compile but panics if the expression cannot be parsed.

Expected behavior

Print a message that points to MulticastInterfaces and includes error from regexp.Compile. Then either exit or ignore the erroneous entry.

shibe2 avatar Nov 09 '23 07:11 shibe2

This looks like expected behavior: almost all config errors cause panic

Slex avatar May 14 '24 23:05 Slex

almost all config errors cause panic

Gross.

shibe2 avatar May 15 '24 19:05 shibe2