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

[PLAT-1079] apply meta_data_filters to referer #460

Closed
wants to merge 5 commits into from

Conversation

tremlab
Copy link

@tremlab tremlab commented May 5, 2018

apply metadata filters to referer and headers Referer to redact any values that should be filtered from query params (identical to url cleaning). Added test for redaction in url, referer and headers Referer.

@tremlab tremlab changed the title apply meta_data_filters to referer [PLAT-1079] apply meta_data_filters to referer May 8, 2018
@Cawllec Cawllec changed the base branch from master to next May 9, 2018 15:48
@Cawllec
Copy link
Contributor

Cawllec commented May 9, 2018

Looks good, I've switched the target branch to next, and there are a couple of fixes for the Rack middleware tests that should be merged from there to this branch. If you could do that it'll be easier to tell if the requests pass.

kattrali pushed a commit that referenced this pull request May 17, 2018
@kattrali
Copy link
Contributor

Moved to #468, did a bit of commit pruning.

@kattrali kattrali closed this May 17, 2018
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.

3 participants