Как победить ошибку «OverflowError: signed integer is greater than maximum»?

Question

[pcdesign]

Вот документация по flask-mongoengine
docs.mongoengine.org/projects/flask-mongoengine/en...

И там есть параграф с паджинацией:

paginated_todos = Todo.objects.paginate(page=page, per_page=10)

Все отлично работает, на страницы разбивается.
Но, если ввести какое-нибудь здоровенное число для page (например, 220000000000), то получим ошибку

OverflowError: signed integer is greater than maximum

Если еще больше, page=22000000444444444444444444444440000, то ошибка
Python int too large to convert to C long

Вопрос. Можно ли как-то обойтись без

try:
     paginated_todos = Todo.objects.paginate(page=page, per_page=10)
except:
     abort(404)

?
Чтобы каждый раз не оборачивать в трай разбивку на страницы?

P.S.

Flask-SQLAlchemy - оказывается тем же самым страдает.

Comments

[Ростислав Григорьев]

Валидировать число page. Например, используя функции min и max

[pcdesign]

Ростислав Григорьев, что валидировать в каждом роуете, что засовывать в try - от перестановки мест слагаемых кол-во кода не изменяется :)

[Ростислав Григорьев]

> валидировать в каждом роуете

дублировать код, конечно не надо. достаточно написать одну функцию

[pcdesign]

Ростислав Григорьев, да видимо и придется делать вариант с min и max. Странно, что это по умолчанию не включено не в mongoengine ни в SQLAlchemy.
Видимо все забивают на это.

Answer 1

[Руслан .]

Можно написать функцию по аналогии с приведенной doc-ой:
# Paginate through todo
def view_todos(page=1):
paginated_todos = Todo.objects.paginate(page=page, per_page=10)

И внутри этой функции делать проверку на наличие ошибки
def view_todos(page=1):
try:
return Todo.objects.paginate(page=page, per_page=10)
except:
abort(404)

А дальше везде в вашем коде использовать уже функцию view_todos, тогда код клонировать не придется.
По сути получается декоратор к функции пагинации.

Comments

[pcdesign]

Спасибо за ответ, но в каждом роуте свои условия.
Например,

Todo.objects(category_id=Cat.id).paginate(page=page, per_page=10)

То есть в эту функцию каждый раз придется передавать еще тьму всяких условий.

[Руслан .]

Можно попробовать использовать распаковку аргументов:
def view_todos(page=1, **kwargs):
try:
return Todo.objects(**kwargs).paginate(page=page, per_page=10)
except:
abort(404)

В таком виде декоратор будет более универсальным, вариант запуска с вашим кейсом:
view_todos(1, category_id=Cat.id )