В биллинге нашли код с комментом «они и так без денег, не трогай»

История о том, как мы проводили аудит кодовой базы перед сделкой с крупной компанией в прошлом году. Тогда покупатель дотошный попался, затребовал полный анализ легаси. Проект живет с 2015 года, так что слоев там накопилось прилично.

В общем, один из сеньоров копался в старом модуле биллинга, где была довольно древняя логика повторных списаний, триалы и прочее. И скидывает мне в личку скрин файла, который, судя по всему, с 2017 никто не открывал.

Там кусок кода, который должен был регулярно запускаться и пытаться до победного списывать деньги с карт, заблокированных банками. Блокировка обычно происходит при аресте счетов, диких долгах или банкротстве. Так вот, этот метод был тупо закомментирован. А чуть выше оставлен комментарий разработчика:

То есть, кто-то втихую выключил функцию и никому не сказал.

В git blame имя автора уже не билось, ибо репозиторий за эти годы трижды переезжал, и старые авторы затерлись. Мы ради интереса полезли вглубь через git log и по хэшу коммита нашли автора. Реально, был такой джун-разработчик, коллега. Посидел в компании полгода в 2017 на удаленке и уволился.

Если интересно, то сделка в итоге прошла нормально, этот легаси-хлам мы все равно под нож пустили при интеграции. Но я до сих пор иногда вспоминаю этого чувака. Сидел, за копейки писал код для крупного сервиса, посмотрел на задачу и решил: «Нет, вот этой херней я заниматься не буду».