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.

Sign up for GitHub

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

Jump to bottom

Sanitize arrays #145

Closed
greg0ire opened this issue Apr 14, 2014 · 4 comments
Closed

Sanitize arrays #145

greg0ire opened this issue Apr 14, 2014 · 4 comments

Comments

@greg0ire
Copy link
Contributor

When a credit card number is posted as an array, sanitizing would be nice. I'll post a failing test case shortly.
@greg0ire
Copy link
Contributor Author

See #147

greg0ire pushed a commit to greg0ire/raven-php that referenced this issue Apr 14, 2014
add failing tests case for getsentry#145
bf0fb33
dcramer added a commit that referenced this issue Apr 14, 2014
@dcramer
Merge pull request #147 from greg0ire/failing_test_case
994e243 
add failing tests case for #145
@dcramer
Copy link
Member

dcramer commented Apr 14, 2014

Dont have time to do this right now, but I'm +1 on this and have merged in the failing test

@matthewgoslett matthewgoslett mentioned this issue May 15, 2015
Closed
@dcramer
Copy link
Member

dcramer commented Apr 28, 2016

Bumping this to ensure were not having any issues in latest release

@Jean85 Jean85 mentioned this issue Jun 21, 2018
Merged
@stayallive
Copy link
Collaborator

This looks to be fixed in 2.0 by #622.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@dcramer @greg0ire @stayallive