feat: add per-call endpoint override config with backward compat - #1691
feat: add per-call endpoint override config with backward compat#1691aritro2002 wants to merge 8 commits into
Conversation
Changed Files
|
🚫 Missing Linked IssueHi 👋 This pull request does not appear to be linked to any open issue yet. Linking your PR to an issue helps keep the project tidy and ensures the issue is closed automatically. ✔️ How to fix this
Once linked, this check will pass automatically on your next push or when you re-run the workflow. Thanks for helping maintainers! 🙌 |
🚫 Missing Linked IssueHi 👋 This pull request does not appear to be linked to any open issue yet. Linking your PR to an issue helps keep the project tidy and ensures the issue is closed automatically. ✔️ How to fix this
Once linked, this check will pass automatically on your next push or when you re-run the workflow. Thanks for helping maintainers! 🙌 |
2 similar comments
🚫 Missing Linked IssueHi 👋 This pull request does not appear to be linked to any open issue yet. Linking your PR to an issue helps keep the project tidy and ensures the issue is closed automatically. ✔️ How to fix this
Once linked, this check will pass automatically on your next push or when you re-run the workflow. Thanks for helping maintainers! 🙌 |
🚫 Missing Linked IssueHi 👋 This pull request does not appear to be linked to any open issue yet. Linking your PR to an issue helps keep the project tidy and ensures the issue is closed automatically. ✔️ How to fix this
Once linked, this check will pass automatically on your next push or when you re-run the workflow. Thanks for helping maintainers! 🙌 |
| let overrideCustomAssetsEndpoint = getFieldStr("overrideCustomAssetsEndpoint") | ||
| let overrideCustomSDKConfigEndpoint = getFieldStr("overrideCustomSDKConfigEndpoint") | ||
| let overrideCustomConfirmEndpoint = getFieldStr("overrideCustomConfirmEndpoint") | ||
| let overrideCustomAirborneEndpoint = getFieldStr("overrideCustomAirborneEndpoint") |
There was a problem hiding this comment.
Please remove airborneEndpoint related logic
There was a problem hiding this comment.
remove overrideCustomAirborneEndpoint as well
|
[should-fix] Please clear |
… feat/custom-endpoint-config-overrides
🚫 Missing Linked IssueHi 👋 This pull request does not appear to be linked to any open issue yet. Linking your PR to an issue helps keep the project tidy and ensures the issue is closed automatically. ✔️ How to fix this
Once linked, this check will pass automatically on your next push or when you re-run the workflow. Thanks for helping maintainers! 🙌 |
🚫 Missing Linked IssueHi 👋 This pull request does not appear to be linked to any open issue yet. Linking your PR to an issue helps keep the project tidy and ensures the issue is closed automatically. ✔️ How to fix this
Once linked, this check will pass automatically on your next push or when you re-run the workflow. Thanks for helping maintainers! 🙌 |
🚫 Missing Linked IssueHi 👋 This pull request does not appear to be linked to any open issue yet. Linking your PR to an issue helps keep the project tidy and ensures the issue is closed automatically. ✔️ How to fix this
Once linked, this check will pass automatically on your next push or when you re-run the workflow. Thanks for helping maintainers! 🙌 |
🚫 Missing Linked IssueHi 👋 This pull request does not appear to be linked to any open issue yet. Linking your PR to an issue helps keep the project tidy and ensures the issue is closed automatically. ✔️ How to fix this
Once linked, this check will pass automatically on your next push or when you re-run the workflow. Thanks for helping maintainers! 🙌 |
… feat/custom-endpoint-config-overrides
Type of Change
Description
Introduces a structured
customConfigendpoint override system, fixing missing wiring foroverrideCustomAssetsEndpointandoverrideCustomConfirmEndpoint, and adding fullparent→child iframe forwarding for all endpoint types. Legacy
customBackendUrlcontinuesto work unchanged.
ref: https://github.com/sh-iv-am/integrations
Files Changed
src/Utilities/ApiEndpoint.resbackendOverrideEndPoint,confirmOverrideEndPoint,assetsEndPoint,sdkConfigEndPoint,loggingOverrideEndPoint,airborneEndPoint), their setters, and getters with documented priority chainssrc/Utilities/S3Utils.resgetBaseUrlnow callsApiEndpoint.getAssetsEndPoint()instead of hardcodedGlobalVars.sdkUrlsrc/Utilities/APIHelpers/APIUtils.resFetchSdkConfigsnow usesgetSdkConfigEndPoint()separately from payment API callssrc/Utilities/Utils.resgetHeadersinjectsx-platform-api-keyheader whenplatformPublishableKeyis setsrc/hyper-log-catcher/HyperLogger.resgetLoggingEndPoint()instead of hardcodedGlobalVars.logEndpointsrc/hyper-loader/Hyper.rescustomConfigfields andplatformPublishableKeyfrom init argssrc/hyper-loader/Elements.resconfirmEndpoint,loggingEndpoint,assetsEndpoint,platformPublishableKeyin parent→child postMessagesrc/hyper-loader/PaymentMethodsManagementElements.ressrc/LoaderController.resconfirmEndpoint,loggingEndpoint,assetsEndpoint,platformPublishableKeyfrom parent messagesrc/hyper-loader/UpdateIntentHelpersNew.resplatformPublishableKeythrough to the preMountLoader iframe URLsrc/Payments/PreMountLoader.resplatformPublishableKeybefore eager API calls firesrc/App.resplatformPublishableKeyfrom query params and passes toPreMountLoaderPriority Matrix (all combinations)
customBackendUrlcustomEndpointoverrideCustomBackendoverrideCustomConfirmoverrideCustomAssetsENV_BACKEND_URLENV_CONFIRM_URLsdkUrlENV_BACKEND_URLcustomBackendUrlcustomBackendUrlcustomBackendUrlcustomBackendUrlcustomEndpoint(wins)customEndpointcustomEndpointcustomEndpointcustomEndpointcustomEndpointcustomEndpointcustomEndpointoverrideCustomBackendcustomEndpointcustomEndpointcustomEndpointcustomEndpointoverrideCustomConfirmcustomEndpointcustomEndpointcustomEndpointcustomEndpointoverrideCustomAssetscustomEndpointoverrideCustomBackendoverrideCustomConfirmoverrideCustomAssetscustomEndpointoverrideCustomBackendENV_CONFIRM_URLsdkUrlENV_BACKEND_URLENV_BACKEND_URLoverrideCustomConfirmsdkUrlENV_BACKEND_URLENV_BACKEND_URLENV_CONFIRM_URLoverrideCustomAssetsENV_BACKEND_URLEndpoint Resolution Reference
How did you test it?
I have tested it locally
Checklist
npm run re:build