Skip to content

Conversation

@bartes
Copy link
Contributor

@bartes bartes commented Mar 23, 2020

No description provided.

@bartes bartes self-assigned this Mar 23, 2020
@bartes bartes marked this pull request as ready for review March 23, 2020 15:48
@bartes bartes requested review from afterdesign, dawlib and nijikon and removed request for nijikon March 23, 2020 15:48
Copy link

@dawlib dawlib left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I would add some more docs though

@bartes bartes merged commit 1a87e8e into master Mar 26, 2020
bartes added a commit that referenced this pull request Mar 26, 2020
* added autopep8 config

* improved headers formatter

* added trusted proxies and formatted ip_headers, updated readme

* removed not used imports

* added missing events

* refactored headers management, added ip_headers defaults, extract ip from headers

* linter fixes and improved formatting

* changelog update
@dawlib dawlib deleted the improve_ip_headers branch March 26, 2020 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants