Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Panic when MulticastInterfaces/Regex has errors. #1101

Open
shibe2 opened this issue Nov 9, 2023 · 2 comments
Open

Panic when MulticastInterfaces/Regex has errors. #1101

shibe2 opened this issue Nov 9, 2023 · 2 comments

Comments

@shibe2
Copy link

shibe2 commented Nov 9, 2023

Current behavior

Regex: regexp.MustCompile(intf.Regex),

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.

@Slex
Copy link

Slex commented May 14, 2024

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

@shibe2
Copy link
Author

shibe2 commented May 15, 2024

almost all config errors cause panic

Gross.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants