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

Update parameter filter logging guides [ci-skip] #44388

Merged
merged 1 commit into from Feb 10, 2022

Conversation

p8
Copy link
Member

@p8 p8 commented Feb 10, 2022

Summary

Instead of just [:password], the filter_parameters configuration includes
a list of filters in the latest filter_parameter_logging initializer template.
This updates the guides to reflect those changes.


Before

image

After

image


Before

image

After

image

@rails-bot rails-bot bot added the docs label Feb 10, 2022
The `filter_parameters` configuration includes a list of filters in the
latest `filter_parameter_logging` initializer template.
This updates the guides to reflect those changes.
@p8 p8 force-pushed the guides/filter-parameters-default branch from 9d6307a to e3beba1 Compare February 10, 2022 14:39
@kamipo kamipo merged commit e42e60e into rails:main Feb 10, 2022
@p8 p8 deleted the guides/filter-parameters-default branch February 10, 2022 16:29
@p8
Copy link
Member Author

p8 commented Feb 10, 2022

Thanks @kamipo !

jonathanhefner added a commit that referenced this pull request Feb 10, 2022
jonathanhefner pushed a commit to jonathanhefner/rails that referenced this pull request Feb 14, 2022
Update parameter filter logging guides [ci-skip]

(cherry picked from commit e42e60e)
jonathanhefner added a commit to jonathanhefner/rails that referenced this pull request Feb 14, 2022
Follow-up to rails#44388.

(cherry picked from commit 0f36425)
public-rant pushed a commit to opensource-rant/rails that referenced this pull request Feb 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants