Skip to content

Migrated the FedEx carrier from SOAP to the REST API #1193

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
fballiano merged 10 commits into from
Aug 11, 2026
Merged

Migrated the FedEx carrier from SOAP to the REST API #1193

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
ab012d2
Migrated the FedEx carrier from SOAP to the REST API
fballiano Jul 30, 2026
7110435
lint
fballiano Jul 30, 2026
57010b6
Verified FedEx label creation and made a refused rollback report failure
fballiano Jul 30, 2026
61aa674
Merge branch 'main' into fedex-rest-migration
fballiano Aug 1, 2026
a4c8ffd
Merge branch 'main' into fedex-rest-migration
fballiano Aug 11, 2026
d2f8c73
Fixed review findings: OAuth token handling, error reporting, per-pac…
fballiano Aug 11, 2026
5b8c6e9
Restored the declared value on SmartPost rate requests, capped at the…
fballiano Aug 11, 2026
993e676
Revert "Restored the declared value on SmartPost rate requests, cappe…
fballiano Aug 11, 2026
1eb16bc
Removed dead tracking request state, duplicate debug logging, and per…
fballiano Aug 11, 2026
bc94060
Fixed shipment response parsing tests to carry a package reference in…
fballiano Aug 11, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions .env.testing.dist
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,5 +4,12 @@
PAYPAL_SANDBOX_CLIENT_ID=
PAYPAL_SANDBOX_CLIENT_SECRET=

FEDEX_SANDBOX_CLIENT_ID=
FEDEX_SANDBOX_CLIENT_SECRET=
FEDEX_SANDBOX_ACCOUNT=
# Leave empty for the standard Rate API; set to "comprehensive" only if your FedEx
# project is registered as an Integrator Provider.
FEDEX_SANDBOX_RATE_ENDPOINT=

# Fixer API access key. Without it the live currency rate import tests skip.
FIXERIO_API_KEY=
8 changes: 8 additions & 0 deletions .github/workflows/pest.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -200,6 +200,10 @@ jobs:
API_BASE_URL: http://127.0.0.1:8080
PAYPAL_SANDBOX_CLIENT_ID: ${{ secrets.PAYPAL_SANDBOX_CLIENT_ID }}
PAYPAL_SANDBOX_CLIENT_SECRET: ${{ secrets.PAYPAL_SANDBOX_CLIENT_SECRET }}
FEDEX_SANDBOX_CLIENT_ID: ${{ secrets.FEDEX_SANDBOX_CLIENT_ID }}
FEDEX_SANDBOX_CLIENT_SECRET: ${{ secrets.FEDEX_SANDBOX_CLIENT_SECRET }}
FEDEX_SANDBOX_ACCOUNT: ${{ secrets.FEDEX_SANDBOX_ACCOUNT }}
FEDEX_SANDBOX_RATE_ENDPOINT: ${{ secrets.FEDEX_SANDBOX_RATE_ENDPOINT }}
FIXERIO_API_KEY: ${{ secrets.FIXERIO_API_KEY }}
run: ./vendor/bin/pest --display-errors --display-warnings

Expand Down Expand Up @@ -324,6 +328,10 @@ jobs:
API_BASE_URL: http://127.0.0.1:8080
PAYPAL_SANDBOX_CLIENT_ID: ${{ secrets.PAYPAL_SANDBOX_CLIENT_ID }}
PAYPAL_SANDBOX_CLIENT_SECRET: ${{ secrets.PAYPAL_SANDBOX_CLIENT_SECRET }}
FEDEX_SANDBOX_CLIENT_ID: ${{ secrets.FEDEX_SANDBOX_CLIENT_ID }}
FEDEX_SANDBOX_CLIENT_SECRET: ${{ secrets.FEDEX_SANDBOX_CLIENT_SECRET }}
FEDEX_SANDBOX_ACCOUNT: ${{ secrets.FEDEX_SANDBOX_ACCOUNT }}
FEDEX_SANDBOX_RATE_ENDPOINT: ${{ secrets.FEDEX_SANDBOX_RATE_ENDPOINT }}
FIXERIO_API_KEY: ${{ secrets.FIXERIO_API_KEY }}
run: ./vendor/bin/pest --display-errors --display-warnings

Expand Down
32 changes: 1 addition & 31 deletions .phpstan.dist.baseline.neon
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21027,24 +21027,6 @@ parameters:
count: 1
path: app/code/core/Mage/Usa/Model/Shipping/Carrier/Dhl/International.php

-
rawMessage: Access to an undefined property Mage_Usa_Model_Shipping_Carrier_Fedex::$_rawTrackingRequest.
identifier: property.notFound
count: 1
path: app/code/core/Mage/Usa/Model/Shipping/Carrier/Fedex.php

-
rawMessage: 'Method Mage_Usa_Model_Shipping_Carrier_Fedex::_getXMLTracking() has no return type specified.'
identifier: missingType.return
count: 1
path: app/code/core/Mage/Usa/Model/Shipping/Carrier/Fedex.php

-
rawMessage: 'Method Mage_Usa_Model_Shipping_Carrier_Fedex::_parseTrackingResponse() has no return type specified.'
identifier: missingType.return
count: 1
path: app/code/core/Mage/Usa/Model/Shipping/Carrier/Fedex.php

-
rawMessage: 'Method Mage_Usa_Model_Shipping_Carrier_Fedex::_setFreeMethodRequest() has no return type specified.'
identifier: missingType.return
Expand All @@ -21057,22 +21039,10 @@ parameters:
count: 1
path: app/code/core/Mage/Usa/Model/Shipping/Carrier/Fedex.php

-
rawMessage: 'Method Mage_Usa_Model_Shipping_Carrier_Fedex::setTrackingReqeust() has no return type specified.'
identifier: missingType.return
count: 1
path: app/code/core/Mage/Usa/Model/Shipping/Carrier/Fedex.php

-
rawMessage: 'Parameter #1 $value of method Mage_Shipping_Model_Rate_Result_Method::setCost() expects float, string given.'
identifier: argument.type
count: 1
path: app/code/core/Mage/Usa/Model/Shipping/Carrier/Fedex.php

-
rawMessage: 'Parameter #1 $value of method Mage_Shipping_Model_Rate_Result_Method::setMethodTitle() expects string, array|bool given.'
identifier: argument.type
count: 2
count: 1
path: app/code/core/Mage/Usa/Model/Shipping/Carrier/Fedex.php

-
Expand Down
Loading
Loading