Skip to content

(#57 Update) Show real remote IP when using reverse proxy (e.g. nginx, caddy...) #296

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.

Sign up for GitHub

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

Open
ar3ph wants to merge 8 commits into
base: master
Choose a base branch
from
Open

(#57 Update) Show real remote IP when using reverse proxy (e.g. nginx, caddy...) #296

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
c8e7032
Real remote IP shown while using proxy
gidantribal Aug 29, 2014
fc6393d
Added ipware depencency
gidantribal Aug 29, 2014
bcb9304
Fixed imports, shame on me
gidantribal Aug 29, 2014
aa96121
Accept local/non routable IPs
gidantribal Aug 29, 2014
d183c2c
merge master
ar3ph Dec 6, 2025
69b9a1b
* Use request.settings.IP_DUMMY for unknown IPs.
ar3ph Dec 6, 2025
3299e2e
remove extra comment
ar3ph Dec 6, 2025
28f0081
fix typo
ar3ph Dec 6, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 6 additions & 2 deletions request/models.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
from django.db import models
from django.utils import timezone
from django.utils.translation import gettext_lazy as _
from ipware.ip import get_ip

from . import settings as request_settings
from .managers import RequestManager
Expand Down Expand Up @@ -70,8 +71,11 @@ def from_http_request(self, request, response=None, commit=True):
self.is_secure = request.is_secure()
self.is_ajax = request_is_ajax(request)

# User information.
self.ip = request.META.get("REMOTE_ADDR", "")
# User infomation

# @see: https://github.com/kylef/django-request/issues/54
self.ip = get_ip(request) if not None else request_settings.IP_DUMMY

self.referer = request.META.get("HTTP_REFERER", "")[:255]
self.user_agent = request.META.get("HTTP_USER_AGENT", "")[:255]
self.language = request.META.get("HTTP_ACCEPT_LANGUAGE", "")[:255]
Expand Down
1 change: 1 addition & 0 deletions requirements.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
django-ipware>=0.0.8 # @see https://github.com/un33k/django-ipware