-
-
Notifications
You must be signed in to change notification settings - Fork 408
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
Allow override of domains and whitelist in rules #132
Conversation
Ah woops, more conflicts 🙂, I'll get this updated |
…nto lozlow/master
This removes the chance for the _default_ (no rule applied) case to overlap with a user-specified rule
This is ready for review, thanks! |
@lozlow Looks like we worked this one simultaneously. Would you mind to have a look at #133 and see what you think? My implementation is slightly different. It combines address/domain whitelists into one field, and respects the MatchWhitelistOrDomain property instead of having the rule override the global config entirely |
+1 for this. |
Hi, this looks awesome, thank you so much! I've rebased and added some minor tweaks in #169 - does that look to you? |
Merged in #169 - thanks again! |
Builds on the work of mathcantin in #63
Closes #57
Closes #63