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

Fix import of parameter filters from Rails #146

Merged
merged 2 commits into from
Aug 23, 2014
Merged

Fix import of parameter filters from Rails #146

merged 2 commits into from
Aug 23, 2014

Conversation

tamird
Copy link
Contributor

@tamird tamird commented Aug 23, 2014

Fixes #137 and addresses part of #139 (needs README update)

@tamird tamird changed the title Filter insanity Fix import of parameter filters from Rails Aug 23, 2014
@JackDanger
Copy link

Excellent. This prevents filtering a parameter called, say, 'q' and having it filter out any parameter with a 'q' in the name. Great work.

@loopj
Copy link
Contributor

loopj commented Aug 23, 2014

Thanks @tamird! This looks great.

loopj added a commit that referenced this pull request Aug 23, 2014
Fix import of parameter filters from Rails
@loopj loopj merged commit 76a316e into bugsnag:master Aug 23, 2014
@tamird tamird deleted the filter-insanity branch August 23, 2014 19:26
@loopj
Copy link
Contributor

loopj commented Aug 23, 2014

Released as 2.4.0 to rubygems, thanks again!

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.

parameter filtering is inconsistent with rails
3 participants