Seams::transport() folds a misconfigured binding into "nothing bound"

Open Beginner friendly
#2 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
68/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Active
Tech stack
php
Domain
backend

Research direction

Start in Support/Seams.php and inspect the transport resolution path where App::make($configured) is checked. Trace the caller that records NoTransportBound, then distinguish a wrong-type binding from no binding while preserving the existing no-binding behavior. Done means the two operator problems produce different refusal reasons.

Written by the indexing model from the issue text.

Description

Support/Seams.php returns null when App::make($configured) resolves to something that is not a DeliveryTransport. The caller then records NoTransportBound, which says nobody answered — but somebody did, and answered with the wrong type.

A host that binds the wrong class reads a log saying it bound nothing, and will look in the wrong place. These are two different operator problems and they deserve two different refusal reasons.

Dominant language
PHP
Stars
0
Forks
0
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from liberusoftware/module-ecommerce-commerce-extensions

All issues in liberusoftware/module-ecommerce-commerce-extensions

Similar issues

More PHP issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.