Skip to content

Отсутствие корректной обработки ответа attachment.not.ready при отправке сообщения #22

@LyohaLyoha

Description

@LyohaLyoha

Если было отправлено сообщение с токеном только что загруженным видео или картинкой в attachment - то иногда апи может вернуть 400 с кодом ошибки "code":"attachment.not.ready"

Сейчас вы просто выплевываете эту ошибку наружу в виде generic MaxApiException, причем в ответе апи есть два поля - code и message, и в MaxApiException вы сделали поля errorCode и errorMessage.

Но в обработке ответа вы забили хер на все это, и просто кидаете MaxApiException, кладя в errorMessage весь пришедший от апи json-string со всеми этими полями.

Хотя по нормальному, ну если проект сделан не для галочки при распиле и откате, и не сдвг-дегенератами - то не только значения по полям должны быть разложены, но еще и собственный класс исключения должен быть, скажем AttachmentNotReadyException.
К сожалению это явно не ваш случай, не смотря на то что у вас "полная совместимость с апи, покрытая тестами".

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions