Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces a major refactor to the SDK's HTTP transport layer, decoupling the SDK from specific HTTP client libraries and enabling flexible transport injection. The changes implement a new abstraction for HTTP communication, update the client and SDK classes to use this abstraction, and provide examples for integrating with multiple HTTP clients (
requests,httpx, andurllib3). The project dependency configuration is also updated to support optional transports.Transport Layer Abstraction and Integration
HTTPTransportandHTTPResponseprotocol interfaces insrc/multisafepay/transport/http_transport.pyto abstract HTTP communication and response handling, enabling the SDK to work with any compatible HTTP client.ClientandSdkclasses to accept atransportparameter instead of a concrete HTTP client, defaulting toRequestsTransportif none is provided. All request logic now uses the transport abstraction. [1] [2] [3] [4] [5] [6] [7]Example Implementations for Multiple HTTP Clients
httpxandurllib3, demonstrating how to inject custom transport implementations and adapt responses to the SDK interface. Therequest_transport.pyexample shows advancedrequests.Sessionusage with retry and connection pooling. [1] [2] [3]Dependency Management and Configuration
pyproject.tomlto moverequestsandurllib3to an optional extra, allowing users to install only the transports they need. Development dependencies now includepython-dotenv,requests,urllib3, andhttpxfor example and test support. [1] [2]Module Structure and Exports
src/multisafepay/transport/__init__.pyto define the transport module's public API and ensure proper import and discoverability.src/multisafepay/__init__.pyto clarify exports.These changes make the SDK more flexible, testable, and extensible for users who want to use different HTTP clients or custom transport logic.