-
Notifications
You must be signed in to change notification settings - Fork 386
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.
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
Add support for PSR-18 HTTP clients #826
base: master
Are you sure you want to change the base?
Add support for PSR-18 HTTP clients #826
Commits on Nov 6, 2023
-
Revert "Merge pull request simplepie#777 from Art4/add-psr18-http-cli…
…ent-support" There are correctness issues as well as disorganized commit history. The fixed changes are re-applied in the commits that follow.
Configuration menu - View commit details
-
Copy full SHA for 9400f7d - Browse repository at this point
Copy the full SHA 9400f7dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 802d33a - Browse repository at this point
Copy the full SHA 802d33aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 58d812a - Browse repository at this point
Copy the full SHA 58d812aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 1092570 - Browse repository at this point
Copy the full SHA 1092570View commit details -
Configuration menu - View commit details
-
Copy full SHA for b0427c9 - Browse repository at this point
Copy the full SHA b0427c9View commit details -
SimplePieTest: materialize provider
Providers are meant for data parametrization. This one basically created a completely different test situation and what we are going to test next will be even more wild. Let’s dissolve the provider into individual test cases.
Configuration menu - View commit details
-
Copy full SHA for 44cdd6b - Browse repository at this point
Copy the full SHA 44cdd6bView commit details -
Set SimplePie->$file to private
We are going to allow using HTTP Clients that will set this to a different `Response` subclasses than just `File`. It was already marked private with PHPDoc, let’s enforce it. Hopefully, this will produce a more scrutable error message in case someone wants to access `File` properties on this.
Configuration menu - View commit details
-
Copy full SHA for 7d64ada - Browse repository at this point
Copy the full SHA 7d64adaView commit details -
Add SimplePie::set_http_client(), add tests
This will allow using PSR-18 HTTP Clients. Since those return a different `Response` subclass than `File`, we need to convert `Response` to `File` to pass it to `Locator`, `Sniffer` & co.
Configuration menu - View commit details
-
Copy full SHA for eddf1a1 - Browse repository at this point
Copy the full SHA eddf1a1View commit details -
Configuration menu - View commit details
-
Copy full SHA for 55bb279 - Browse repository at this point
Copy the full SHA 55bb279View commit details -
Configuration menu - View commit details
-
Copy full SHA for 67528cc - Browse repository at this point
Copy the full SHA 67528ccView commit details -
Configuration menu - View commit details
-
Copy full SHA for 089fe8f - Browse repository at this point
Copy the full SHA 089fe8fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 1f2881a - Browse repository at this point
Copy the full SHA 1f2881aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 35788eb - Browse repository at this point
Copy the full SHA 35788ebView commit details -
Add Sanitize::set_http_client()
Allows passing PSR implementations to Sanitize instead of internal HTTP Client. TODO: Possibly make the set_http_client method internal again.
Configuration menu - View commit details
-
Copy full SHA for 6a188f8 - Browse repository at this point
Copy the full SHA 6a188f8View commit details -
Add Locator::set_http_client()
Allows passing PSR implementations to Locator instead of internal HTTP Client. TODO: Possibly make the set_http_client method internal again.
Configuration menu - View commit details
-
Copy full SHA for 0427439 - Browse repository at this point
Copy the full SHA 0427439View commit details