Skip to content

Conversation

@anderly
Copy link
Member

@anderly anderly commented Sep 4, 2025

  • Updated IHttpProvider interface to extend PSR-18 ClientInterface
  • Added PSR-7/17/18 dependencies to composer.json
  • Implemented sendRequest() method in GuzzleHttpProvider and Psr17HttpProvider
  • Created HttpRequestBuilder for PSR-7 request conversion
  • Added HttpClientException for PSR-18 compliant error handling
  • Maintained backward compatibility with existing send() method
  • Fixed ODataRequest to use correct send() method
  • All tests passing without modification

This change enables the use of any PSR-18 compliant HTTP client with the OData client library while maintaining full backward compatibility.

@anderly anderly changed the title feat: implement PSR-17 and PSR-18 standards for HTTP providers Implement PSR-17 and PSR-18 standards for HTTP providers Sep 4, 2025
- Updated IHttpProvider interface to extend PSR-18 ClientInterface
- Added PSR-7/17/18 dependencies to composer.json
- Implemented sendRequest() method in GuzzleHttpProvider and Psr17HttpProvider
- Created HttpRequestBuilder for PSR-7 request conversion
- Added HttpClientException for PSR-18 compliant error handling
- Maintained backward compatibility with existing send() method
- Fixed ODataRequest to use correct send() method
- All tests passing without modification

This change enables the use of any PSR-18 compliant HTTP client with the OData client library while maintaining full backward compatibility.

🤖 Generated with Claude Code

Co-Authored-By: Claude <[email protected]>
@anderly anderly force-pushed the feature/psr-17-18-implementation branch from 5c89302 to bf176e0 Compare September 4, 2025 05:12
anderly and others added 3 commits September 4, 2025 00:14
- Add ResponseInterface return type to send() method
- Implement PSR-18 sendRequest() method
- Update mock response to properly implement PSR-7 ResponseInterface
- Add proper type hints to all PSR-7 interface methods

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@anderly anderly merged commit af2a4e6 into master Sep 4, 2025
12 checks passed
@anderly anderly deleted the feature/psr-17-18-implementation branch September 5, 2025 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants