К обсуждениям

Craft Commerce: partial payment строго положителен

Редакция VOne Технологии

Защитная проверка Craft Commerce partial payment по ghsa-78vr-q6cf-c7p6: применимость, безопасный fixture, raw-value / parsed-value / currency-step / validation-result / gateway-calls / order-state, pass-rule и stop-rule без production-данных.

Какая граница сломана в Craft Commerce partial payment

Пользовательская проблема здесь узкая: нулевая или отрицательная paymentAmount проходит controller и влияет на состояние заказа. Механизм по прямой upstream advisory: Float cast не заменяет бизнес-инвариант. Проверка должна выполняться до gateway, transaction и изменения paid status, учитывая finite value и валютную точность. Поэтому проверяем не общий класс уязвимостей, а invariant «payment amount является finite, строго положительным и допустимым для remaining balance до любого side effect». GitHub Reviewed Advisory ghsa-78vr-q6cf-c7p6 опубликована 2026-06-19, обновлена 2026-06-19; package boundary: «composer/craftcms/commerce >= 5.0.0, <= 5.6.4; first patched 5.6.5». Эти факты не доказывают affected code в конкретном fork, эксплуатацию, популярность или поисковый спрос.

Сначала докажите применимость сборки и пути

Запишите source package, runtime version, build digest, commit provenance, feature/config state и роль, достигающую Craft Commerce partial payment. Отдельно отметьте самый ранний side effect. Нормальный контроль: минимальная допустимая сумма проходит в fake gateway без реального платежа. Если component заменён, feature выключен или backport подтверждён по diff, укажите not-reachable/not-affected с доказательством. Если provenance неизвестна, статус unknown; номер версии из «composer/craftcms/commerce >= 5.0.0, <= 5.6.4; first patched 5.6.5» остаётся фильтром, а не готовым выводом.

Проведите один обратимый лабораторный опыт

Безопасный fixture: В controller unit test передать synthetic amounts -1, 0, NaN, минимальный положительный шаг и сумму выше остатка; gateway заменить counter. Все значения синтетические; persistence — memory adapter или rollback transaction, filesystem — mkdtemp, сеть отключена либо заменена recorder. До запуска сохраните input digest, ожидаемый pass-rule и нулевые counters. После — строку «raw-value / parsed-value / currency-step / validation-result / gateway-calls / order-state», final-state digest и cleanup proof. Эксплуатационный payload, реальный credential, массовый перебор и production traffic не нужны.

Решение по строке raw-value / parsed-value / currency-step / validation-result / gateway-calls / order-state

Сначала убедитесь, что control действительно достиг нужной ветви. Затем читайте «raw-value / parsed-value / currency-step / validation-result / gateway-calls / order-state» слева направо: где нормализован input, когда применена policy и был ли чувствительный action. PASS только если неположительные и non-finite значения отклонены до gateway, order digest не меняется. Отсутствие exception или один HTTP status недостаточны. При async/race boundary используйте фиксированные barriers и один малый повтор, а не нагрузочный тест. Любой необъяснимый side effect переводит результат в fail/unknown до расследования.

Проверьте patch provenance и красные флаги

Исправление обязано реализовать смысл «payment amount является finite, строго положительным и допустимым для remaining balance до любого side effect», а не только изменить номер релиза. На одинаковом fixture сравните affected и candidate build, сохранив raw-value / parsed-value / currency-step / validation-result / gateway-calls / order-state. Красные флаги: policy после side effect, несовпадение verified/effective target, fail-open при ошибке, другой code path для transport/формата или отсутствие negative control. Production update не разрешён этой статьёй: требуется отдельный rollout с backup, canary, readiness, журналами и rollback.

Stop-rule, support packet и дерево решения

Stop-rule: не создавать заказ, не обращаться к платёжной системе и не использовать цену/скидку как CTA. Также остановитесь при privilege prompt, данных вне fixture, необратимой записи, росте ресурсов, отсутствии normal control или невозможности cleanup. В обезличенный support packet положите ghsa-78vr-q6cf-c7p6, product/component, version/build provenance, «composer/craftcms/commerce >= 5.0.0, <= 5.6.4; first patched 5.6.5», строку «raw-value / parsed-value / currency-step / validation-result / gateway-calls / order-state», expected/observed, stop reason и обе source URL. Дерево: patched provenance — not-applicable; недостижимая ветвь — not-reachable; pass-rule выполнен — ready-for-reviewed-update; наблюдается «нулевая или отрицательная paymentAmount проходит controller и влияет на состояние заказа» — fail; иначе unknown. Ни один лист не обещает универсальную безопасность или индексацию.

Материал подготовлен редакцией VOne с помощью ИИ; даты, диапазоны, прямые источники, безопасный fixture, privacy-границы и отсутствие рекламных обещаний затем перепроверены.

Источники и проверка

Информация актуальна на дату публикации. Правила сервисов, приложений и сетей могут меняться.

Ответы

0 опубликовано
Ответов пока нет. Вы можете начать обсуждение.

Ваш ответ

Добавьте свой опыт или уточнение по теме.

Вы публикуете как Аноним Аватар отличает разговоры, но не раскрывает личные данные.

Ответ появится сразу. Не публикуйте личные данные, ключи и приватные ссылки.