Положил прод на полчаса одним дефолтным аргументом. Python который мы заслужили

[lrichard]

Каюсь и делюсь, чтобы хоть кому-то сэкономить нервы. Сервис на FastAPI, поиск по каталогу. В хелпере сигнатура def apply_filters(query, extra=[]). Внутри в одной ветке extra.append(фильтр по региону юзера).

Кто знает, тот уже смеётся. Дефолтный список создаётся один раз и живёт между вызовами. Фильтры копились от запроса к запросу, и юзеры начали получать выдачу с чужими региональными фильтрами. Саппорт завален, мы полчаса ищем призрака, потому что локально не воспроизводится (воркер-то свежий).

Про mutable default я знаю со времён первых туториалов. На собесах сам спрашивал. И всё равно в ревью проскочило. Итог: ruff с правилом B006 теперь в CI блокирующий, без него мердж не проходит.

[sepiatone]

лол, классика жанра. фокусу двадцать лет, он в каждом списке вопросов для джунов, и всё равно регулярно кладёт проды. питон неизменен

[kotlin123]

вопрос вообще не к питону. как append к входному аргументу прошёл ревью? мутировать параметр функции это само по себе красный флаг, хоть с дефолтом хоть без

[wasmsre]

У меня было тоньше: def log_event(payload, ts=datetime.now()). Время вычислилось один раз при импорте модуля. Неделю все события писались одним таймстемпом, аналитики потом неделю восстанавливали отчёты. Никакой линтер тогда не стоял, 2022 год, молодые были.

[rawgoblin]

Для тех кто не понял почему так: дефолты вычисляются один раз в момент def, это документированное поведение, а не баг. Был PEP 671 про late-bound defaults со синтаксисом =>, так и лежит без принятия уже который год, потому что обратная совместимость и споры о синтаксисе. Так что живём как живём.

[ceph7]

полчаса на такое это вы ещё быстро. но ваще канарейка или хотя бы алерт на аномалии в выдаче нашли бы это за пять минут, у вас с обвязкой беда а не с питоном

[mgyorko]

@sepiatone, ruff B006 плюс B008 ставьте парой. B008 как раз ловит вызовы в дефолтах типа того datetime.now() выше, и кстати знает про fastapi Depends чтобы не орать зря.