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_sensitive_data in query params #18

Closed
semanticart opened this issue Mar 15, 2015 · 2 comments
Closed

filter_sensitive_data in query params #18

semanticart opened this issue Mar 15, 2015 · 2 comments

Comments

@semanticart
Copy link

First off: great library, thank you.

I find myself wanting to filter out sensitive data in the query params of a url I'm recording AND still use match_requests_on: [:query]. So basically I want to match on URL but substitute false values for real ones. So I could match a recording with url http://example.com?api_key=FAKE_KEY&page=1 when fetching url http://example.com?api_key=abcdefg&page=1

It would not match a cassette with url http://example.com?api_key=abcdefg&page=SOMETHING_ELSE

Is this feature something you'd adopt? Any advice? I can try to put up a PR, but don't want to waste the time if you wouldn't find it valuable.

@parroty
Copy link
Owner

parroty commented Mar 16, 2015

Thank you for the comment. I haven't tried this case, but I think it makes sense. If you can come up with PR, that's great.

I don't have clear idea how it should be modified yet (need to check back the code more..). I think the query parameter is not well handled for filter_sensitive_data, and just remove or not at the moment (at around ExVCR.Converter). Updating around this might solve the case?

@parroty
Copy link
Owner

parroty commented Aug 1, 2015

It would be addressed by #26 and I'm closing at the moment. If you find any issue, please reopen or open another one.

@parroty parroty closed this as completed Aug 1, 2015
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

No branches or pull requests

2 participants