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

Elixir parameter filtering does not match Phoenix's behavior #936

Closed
matsimitsu opened this issue Apr 24, 2024 · 0 comments · Fixed by #937
Closed

Elixir parameter filtering does not match Phoenix's behavior #936

matsimitsu opened this issue Apr 24, 2024 · 0 comments · Fixed by #937
Assignees
Labels

Comments

@matsimitsu
Copy link
Member

In Phoenix, if you have a param filter for password it will filter any parameter value where the key contains the string password (e.g. including user-password).

The behavior of our own parameter filtering only filters the value if the key exactly matches the key to filter. (e.g. password is filtered, but user-password is not).

Ideally the behavior is the same as Phoenix's behavior.

Intercom: https://app.intercom.com/a/inbox/yzor8gyw/inbox/shared/unassigned/conversation/16410700311389?view=List

@matsimitsu matsimitsu added the bug label Apr 24, 2024
@tombruijn tombruijn self-assigned this Apr 25, 2024
tombruijn added a commit that referenced this issue Apr 25, 2024
The Phoenix Logger filters out partial keys matches. Our implementation
for Phoenix's filter_parameters did not follow this implementation. This
change adds support for filtering out partial key matches when using the
discard filtering strategy.

Phoenix implementation:
https://github.com/phoenixframework/phoenix/blob/dfb0c00d2077e10f8df6cc6e334e04924c4c2bcd/lib/phoenix/logger.ex#L157-L199

Fixes #936
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 a pull request may close this issue.

2 participants