Есть произвольный http сервис и реализован следующий обработчик для регистрации платежей // плательщиком
Опишите что бы вы изменили в коде ниже? Корректно ли все?

Функция paymentsPOST(Запрос) // Запрос - HTTPЗапрос
    Токен = Запрос.Заголовки.Получить("Token");
    Если Токен = Неопределено Тогда
        Возврат Новый HTTPСервисОтвет(400);    
    КонецЕсли;    
    Плательщик = Справочники.Плательщики.НайтиПоРеквизиту("Токен", Токен);
    Если Плательщик.Пустая() Тогда
        Возврат Новый HTTPСервисОтвет(403);    
    КонецЕсли;    
    ЧтениеJSONДанных = Новый ЧтениеJSON;
    РезультатHTTPЗапроса = Запрос.ПолучитьТелоКакСтроку();
    ЧтениеJSONДанных.УстановитьСтроку(РезультатHTTPЗапроса); //Запрос.ПолучитьТелоКакСтроку());
    Структура = ПрочитатьJSON(ЧтениеJSONДанных);
    ЧтениеJSONДанных.Закрыть();     
    Если Структура.Свойство("amount")  = Неопределено Тогда
        Возврат Новый HTTPСервисОтвет(400);    
    КонецЕсли;    
    Попытка
        НовыйПлатеж = Документы.Платеж.СоздатьДокумент();
        НовыйПлатеж.Дата        = ТекущаяДата();
        НовыйПлатеж.Плательщик    = Плательщик;
        НовыйПлатеж.Сумма        = Структура.amount;
        // ...
        НовыйПлатеж.Записать(РежимЗаписиДокумента.Проведение);
    Исключение
        НовыйПлатеж.Записать(РежимЗаписиДокумента.Запись);
        Ответ = Новый HTTPСервисОтвет(400);
        Ответ.УстановитьТелоИзСтроки(ОписаниеОшибки(),КодировкаТекста.UTF8);
        Возврат Ответ;
    КонецПопытки;
    Возврат Новый HTTPСервисОтвет(201);
КонецФункции

Ответ = Новый HTTPСервисОтвет(400);
Ответ.УстановитьТелоИзСтроки(ОписаниеОшибки(),КодировкаТекста.UTF8);

Тут не по феншую. Надо понимать, почему ошибка. И возвращать правильный код. От 403, 404 до 500.

(0)

вот это за гранью добра


Попытка
        НовыйПлатеж = Документы.Платеж.СоздатьДокумент();
        НовыйПлатеж.Дата        = ТекущаяДата();
        НовыйПлатеж.Плательщик    = Плательщик;
        НовыйПлатеж.Сумма        = Структура.amount;
        // ...
        НовыйПлатеж.Записать(РежимЗаписиДокумента.Проведение);
    Исключение


запустил значится это изделие N-раз, слепили N-документов

В ответе, может быть и не json, и
Структура = ПрочитатьJSON(ЧтениеJSONДанных); эта шляпа может в исключение вылететь

Разделить логику передачи данных и работы с объектами.

(0) Коды ошибок надо согласовать с разработчиками этого hhtp-сервиса - возвращать те коды, что они могут и будут обрабатывать.
Добавить описание ошибки, опять же при условии, будут ли они эти ошибки анализировать
Как уже упомянули - добавить проверку, что нужный документ уже был загружен ранее. В этом случае либо ничего не менять, либо сделать перезаполнение существующего документа, а не создавать новый.

(4) Проверку на json делать не обязательно. Если запрос сюда пробился, значит, он уже соответствует API.
А вот завернуть весь обработчик в Попытку-Исключение следует. Чтобы в случае любой нештатной ситуации вернуть ошибку.

(0) переписать нафуй нормально
лапшекод же, неужели сам не видишь

получение данных, обработка данных и отправка/возврат должны быть отделены

(0)
1. Если Токен = Неопределено Тогда
        Возврат Новый HTTPСервисОтвет(400);    
    КонецЕсли;

Добавь в тело запроса ошибку, что нет токена в заголовках

2. чтение JSON оберни в попытку с возвратом ошибки 400 в случае исключения

3. Если ошибка возникла при проведении документа, то возвращай ошибку 500

4. Если запрос выполнился, то в теле запроса верни идентификатор созданного документа

5. лучше в модуле сервиса оставить только код по обработке запроса и формированию ответа, а код по созданию документа вынести в общий модуль или в модуль менеджера/объекта.

(6) > Если запрос сюда пробился, значит, он уже соответствует API.
С фига-ли? Это тебе не SOAP, где схема проверяется.

(8) Первый случай, это вообще 403 в чистом виде

(10) можно и 403, но в теле ответа написать, что нет заголовка "Token". Может сам токен есть, просто не туда засунули

(8) +1

(8) >3. Если ошибка возникла при проведении документа, то возвращай ошибку 500

проводить онлайн в обработке запроса идеологически неправильно
правильно ставить в очередь и возвращать id задания, затем в фоне проводить и отдельный метод получения статуса задания-проведения

(13)+ как id задания логично использовать уид

(13) Только в случае тяжелого проведения. Иначе схлопочешь ситуацию, что затраты на два обращения и побочную запись/чтение в регистр превысят затраты на одно обращение.

(15) лучше сразу продумать и сделать правильно
иначе потом будет очень сложно отделить сервис и сделать чтобы в моменты регламентных операций (обновление конфы и прочее) сервис был доступен 24/7

(16) Хм. А как сделать сервис доступным во время "(обновление конфы и прочее)"?

(17) Через прослойку. В виде отдельной 1с-базы. Которая тоже будет обновляться и прочее.
А чтобы от её обновлений защититься - сделать ещё одну.

(18) Ну про микросервис я догадывался. Но все же?

(19) Две базы и балансер. Обслуживание асинхронное.

Ну ладно, балансер на го, работающий на ардуине.

(20) про nginx не слыхал?

(20) (21) Хм. Примерно так и думал. :-)

(22) вторая база может быть как РИБ копией, так и "микросервисом" накопительным

(16) Если база недоступна в моменты регламентных операций - то это проблемы того, кто к ней обращается.

(24) Если невозможно обеспечить доступность 24/7 следует с http/веб-сервисов переходить на очереди/брокеры сообщений

(25) И если недоступен брокер сообщений, то это все равно проблемы того, кто к нему обращается.

(24) https://cdn.trinixy.ru/pics5/20120313/mail_30.gif

(0) Если Структура.Свойство("amount")  = Неопределено Тогда

так не бывает

(16) > лучше сразу продумать и сделать правильно
Главное, чтобы это "правильно" не было оверинженирингом.
Асинхронность добавляет сложности и добавляет неудобств. Поэтому сразу пихать ее во все дыры на случай когда через сто лет это окупится, если компания в 200 раз вырастет - такое себе.

На практике удобно и чаще всего достаточно делать на синхронных, но быстрых операциях. По возможности без блокировок. Тяжелые проведения ессно не очень разумно выполнять синхронно. Но и это может быть допустимо. От ситуации зависит. В качестве компромисса иногда допустимо синхронно писать, а проводить уже очередью.