должен делать?
он должен выполнить либо переданную задачу, либо вернуть ошибку, если не успел уложить в 2 минуты
А забросить токен внутрь super function не?
я тоже до этого дошла, но ошибку с "slow.. " он не кидает, то есть по идее изначально проблема не в этом
? Этот код оправдан тогда, когда над super function нет контроля. И то, с замечаниями. Если контроль есть, какой смысл городить проверку cancellation token снаружи, а не внутри?
ну этот cancellation token именно для функции таймаута создавался. сейчас если таймаут все таки случается, то Super функция посто продолжает чтото делать в фоне, возвращает результат в никуда и заканчивается. да, это можно поправить, но я не смогла воспроизвести ситуацию, где это выбрасывало бы ошибку, которая конкретно в sentry валится, если бы такое было - то была бы еще одна ошибка с текстом "slow.."
Я бы поправил. Лишняя работа - никогда не хорошо, лучше всего код научить падать как надо. Это же кладезь для дудоса - работа, которая не заканчивается по таймауту! =) А касательно непонятного null - по коду на скрине ошибок быть не должно, WaitAny - говнокод, там должен быть WhenAny. Либо ошибка где-то ещё, либо надо снимать дампы / отлаживать нагорячую
хорошо, попробую залить эти правки и дальше уже с ними смотреть наверное, спасибо)
Обсуждают сегодня