add support to handle response #30
Conversation
anaumov
left a comment
anaumov
left a comment
There was a problem hiding this comment.
Спасибо за реквест!
Прошу сделать тексты читаемыми. Из тех, что есть сейчас сложно понять в чем ошибка. Есть подозрение, что их отдает ФНС, а там документы для юристов, а не для обычных людей.
Главный вопрос, чем выброс ошибки, лучше чем возврат статуса?
| # status 400 | ||
| class FnsBadRequestError < APIError | ||
| def message | ||
| 'Не пройден форматно-логический контроль реквизитов фискальных документов' |
There was a problem hiding this comment.
Переформулируйте, плиз, по-человечески. Из этого текста не ясно в чем дело)
| # status 406 | ||
| class FnsCryptoProtectionError < APIError | ||
| def message | ||
| 'Направленный фискальный признак не прошел проверку системы криптозащиты, поиск фискального документа прерван, дальнейшие попытки проверки запрещены' |
There was a problem hiding this comment.
ФП не соответствует фискальным данным. Проверка невозможна.
| # status 503 | ||
| class FnsServiceUnaviableError < APIError | ||
| def message | ||
| 'Недокументированная ошибка в работе сервиса, для выяснения причин следует обратиться в техподдержку, указав: URL запроса к сервису, текст запроса к сервису, текст ответа от сервиса.' |
There was a problem hiding this comment.
Что-то пошло не так, напишите в техподдержку.
Хорошо бы сразу ссылку или почту указать.
There was a problem hiding this comment.
Не уверен с почтой, прямого доступа у меня к ней нет, но у них все таки выдача доступов по запросу, может там разные менеджеры на отдельную интеграцию. Тогда лучше наверное оставить просто как есть, а там уже на месте определяться, к кому обращаться с таким вопросом
|
@fedorkk посмотри, пожалуйста. |
Выброс ошибки мне кажется более наглядным, т.е внутри приложения уже нет прямой необходимости обработку отдельных статусов настраивать. |
|
Я в свое время специлньо оставил ошибки не обработанными, что бы за это отвечал сервис, использующий API. К примеру мне несоклько раз писали разработчики из-за границы, которые не говорят по русски и им будет тяжело работать с русскоязычным текстом ошибок. |
3 participants
Добавил обработку ответов с выбрасыванием исключений