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

add https connection and allowed hosts check #1575

Merged
merged 9 commits into from
Dec 19, 2023
38 changes: 32 additions & 6 deletions website/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@
from django.db.models import Sum, Count, Q
from django.db.models.functions import ExtractMonth
from django.dispatch import receiver
from django.http import Http404,JsonResponse,HttpResponseRedirect,HttpResponse,HttpResponseNotFound
from django.http import Http404,JsonResponse,HttpResponseRedirect,HttpResponse,HttpResponseNotFound, HttpResponseBadRequest
from django.shortcuts import render, redirect, get_object_or_404
from django.template.loader import render_to_string
from django.utils.decorators import method_decorator
Expand Down Expand Up @@ -266,23 +266,49 @@ def newhome(request, template="new_home.html"):
}
return render(request, template, context)

def is_safe_url(url, allowed_hosts, allowed_paths=None):
if not is_valid_https_url(url):
return False

parsed_url = urlparse(url)

if parsed_url.netloc not in allowed_hosts:
return False

if allowed_paths and parsed_url.path not in allowed_paths:
return False

return True

def safe_redirect(url, allowed_hosts, allowed_paths=None):
if is_safe_url(url, allowed_hosts, allowed_paths):
safe_url = rebuild_safe_url(url)
return redirect(safe_url)
else:
return HttpResponseBadRequest('Invalid redirection URL.')

def github_callback(request):
ALLOWED_HOSTS = ['github.com']
params = urllib.parse.urlencode(request.GET)
url = f"{settings.CALLBACK_URL_FOR_GITHUB}?{params}"
parsed_url = urlparse(url)
if parsed_url.netloc in ALLOWED_HOSTS:
return redirect(url)

return safe_redirect(url, ALLOWED_HOSTS)


def google_callback(request):
ALLOWED_HOSTS = ['accounts.google.com']
params = urllib.parse.urlencode(request.GET)
return redirect(f"{settings.CALLBACK_URL_FOR_GOOGLE}?{params}")
url = f"{settings.CALLBACK_URL_FOR_GOOGLE}?{params}"

return safe_redirect(url, ALLOWED_HOSTS)


def facebook_callback(request):
ALLOWED_HOSTS = ['www.facebook.com']
params = urllib.parse.urlencode(request.GET)
return redirect(f"{settings.CALLBACK_URL_FOR_FACEBOOK}?{params}")
url = f"{settings.CALLBACK_URL_FOR_FACEBOOK}?{params}"

return safe_redirect(url, ALLOWED_HOSTS)


class FacebookLogin(SocialLoginView):
Expand Down
Loading