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

Filter params #208

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Filter params #208

wants to merge 2 commits into from

Conversation

hoisie
Copy link
Owner

@hoisie hoisie commented Aug 11, 2016

No description provided.

hoisie added 2 commits August 10, 2016 18:54
Previously, when a new web.Server object was created using the NewServer
method, the main config would be assigned as the server config. Instead,
extract the default config into a variable and use that. This is mainly
to allow server configs to be independent of the main server config if
necessary.
Certain sensitive parameters (e.g passwords) should never be logged.
Add support for a `FilterParams` config option that contains a list of
regular expression strings to be filtered. When a request is made
containing a param with a sensitive name, the value is replaced with the
string `[filtered]`.
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

Successfully merging this pull request may close these issues.

1 participant