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
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions