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

remove unused user agent filter #2338

Closed
wants to merge 2 commits into from

Conversation

butonic
Copy link
Contributor

@butonic butonic commented Dec 8, 2021

I removed the unused allowed_user_agents config option. It isnot used anywhere in the code, but three places had the config option. Dropping it before it spreads further. We can bring it back in a clean way later.

Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
@butonic butonic force-pushed the remove-unused-user-agent-filter branch from 40dad81 to 08348f5 Compare December 8, 2021 08:23
@butonic butonic marked this pull request as ready for review December 8, 2021 10:23
@butonic butonic requested review from glpatcern, ishank011, labkode and a team as code owners December 8, 2021 10:23
@ishank011
Copy link
Contributor

@butonic this is used in the master branch. Please add back the code to the edge branch as this is a required functionality. Context: #2212

@refs refs self-requested a review December 14, 2021 08:46
@butonic butonic closed this Apr 28, 2022
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