-
Notifications
You must be signed in to change notification settings - Fork 483
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
Support regular expressions in allowlist #660
Conversation
Codecov Report
@@ Coverage Diff @@
## master #660 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 19 19
Lines 651 666 +15
=========================================
+ Hits 651 666 +15
Continue to review full report at Codecov.
|
@bpo Thanks for the PR! I really like this change, and with the switch to One last thing before I sign off on it: would you please add a changelog entry? @etagwerker This looks good to me. What do you think? |
👍 added changelog, and another example using a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bpo @botandrose This looks good to me too. Thank you! ❤️
It's common for database connection parameters to be specified as part of the URL (e.g.
DATABASE_URL=postgres://foo.bar?pool=16
). I'd like to be able to manipulate those connection parameters in my test rig without needing to alter the allowlist.This patch supports this by letting allowlists optionally be specified as regular expressions.