mirror of
https://github.com/EDeev/tablo.git
synced 2026-10-07 20:49:31 +03:00
Вход: переход по ?next= только на страницы этого сайта
После входа приложение перенаправляло на любой адрес из параметра next — ссылкой вида /login?next=https://… можно было увести пользователя на чужой сайт.
This commit is contained in:
parent
164623417c
commit
3a21ef916a
1 changed files with 13 additions and 2 deletions
|
|
@ -1,3 +1,5 @@
|
||||||
|
from urllib.parse import urlsplit
|
||||||
|
|
||||||
from flask import Blueprint, render_template, redirect, url_for, flash, request
|
from flask import Blueprint, render_template, redirect, url_for, flash, request
|
||||||
from flask_login import login_user, logout_user, login_required, current_user
|
from flask_login import login_user, logout_user, login_required, current_user
|
||||||
from app import db
|
from app import db
|
||||||
|
|
@ -6,6 +8,13 @@ from app.models.user import User
|
||||||
bp = Blueprint('auth', __name__)
|
bp = Blueprint('auth', __name__)
|
||||||
|
|
||||||
|
|
||||||
|
def _is_safe_next(target: str) -> bool:
|
||||||
|
"""Переход после входа — только на страницы этого же сайта (защита от открытого редиректа)."""
|
||||||
|
parts = urlsplit(target)
|
||||||
|
return (bool(target) and target.startswith('/') and not target.startswith(('//', '/\\'))
|
||||||
|
and not parts.netloc and not parts.scheme)
|
||||||
|
|
||||||
|
|
||||||
@bp.route('/register', methods=['GET', 'POST'])
|
@bp.route('/register', methods=['GET', 'POST'])
|
||||||
def register():
|
def register():
|
||||||
if current_user.is_authenticated:
|
if current_user.is_authenticated:
|
||||||
|
|
@ -42,8 +51,10 @@ def login():
|
||||||
user = User.query.filter_by(login=login).first()
|
user = User.query.filter_by(login=login).first()
|
||||||
if user and user.check_password(password):
|
if user and user.check_password(password):
|
||||||
login_user(user, remember=bool(request.form.get('remember')))
|
login_user(user, remember=bool(request.form.get('remember')))
|
||||||
next_page = request.args.get('next')
|
next_page = request.args.get('next', '')
|
||||||
return redirect(next_page or url_for('schedules.profile'))
|
if not _is_safe_next(next_page):
|
||||||
|
next_page = url_for('schedules.profile')
|
||||||
|
return redirect(next_page)
|
||||||
flash('Неверный логин или пароль', 'danger')
|
flash('Неверный логин или пароль', 'danger')
|
||||||
return render_template('auth/login.html')
|
return render_template('auth/login.html')
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue