Skip to content

Commit d22fa1b

Browse files
authored
Merge pull request #5829 from alphagov/add_headers_documentation
Adding comment around headers highlighting the list of headers docume…
2 parents 2719095 + f9336c3 commit d22fa1b

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

app/event_handlers.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,8 @@ class Events(metaclass=EventsMeta):
5353

5454
# This might not be totally correct depending on proxy setup
5555
def _get_remote_addr():
56+
# If other headers are required (or this one no is longer needed) update the docs:
57+
# https://github.com/alphagov/notifications-manuals/wiki/Request-headers-used
5658
if request.headers.getlist("X-Forwarded-For"):
5759
return request.headers.getlist("X-Forwarded-For")[0]
5860
return request.remote_addr

0 commit comments

Comments
 (0)