Skip to content
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

Jump to bottom

Php vcr master #3

Open
wants to merge 248 commits into
base: master
Choose a base branch
from
Open

Php vcr master #3

wants to merge 248 commits into from

Conversation

Tjeerd
Copy link
Member

@Tjeerd Tjeerd commented Mar 1, 2023

Context

What has been done

How to test

Todo

  • [ ]
  • [ ]

Notes

endorama and others added 30 commits October 18, 2017 14:59
@endorama
correctly get curl info options
9cc8c3f
@endorama
record curl infos in response
5a51a03
@endorama
test restore curl info from storage
4e607fa
@endorama
fix array short syntax, unsupported on php 5.3
018ce65
@renatomefi
Updating curl test to have a representative curl_info
054dc5a
Disabled opcache when intercepting file
db3dd28
@moufmouf
Adding failing test for Opcache
62f62ec
@moufmouf
Merge branch 'fix-opcache' of github.com:ValdoTR/php-vcr into failing…
60fe67b 
…_opcache
@moufmouf
Adding failing unit test on cURL errors
7d4c26c
@moufmouf
Adding PHPStan to CI build
9f8b60c
@moufmouf
Fixing PHPStan level 0 issues
8e725b5
@moufmouf
Adding failing integration test for wrong URL with SoapClient
3535833
@moufmouf
Adding failing integration test with ConnectException.
1ae0d09 
cURL error is not forwarded correctly.
@moufmouf
Forwarding cURL errors from HttpClient to SoapClient and CurlHook
8f56355
@moufmouf
Merge pull request #1 from vitsoe/fix-curl-infos
ab5bc98 
Add recording of curl infos
@moufmouf
Stepping up minimum PHP version to 7.2
69bea33
@moufmouf
Upgrading to PHPUnit 7
4d78bd2
@moufmouf
Merge pull request php-vcr#4 from moufmouf/modernize_phpunit
924162f 
Upgrading to PHPUnit 7
@moufmouf
Removing non existent constant.
084a955
@moufmouf
Fixing PHP-Doc annotation syntax (and reaching PHPStan level 2)
c97f1bb
@moufmouf
Fixing StreamProcessor returns (true on success, false on failure)
765678f
@moufmouf
Adding nullable type hints in PHPDoc for explicitly nullable properti…
0b7fa41 
…es / return types.

PHPStan level 3 reached.
@moufmouf
Adding nullable type hints on nullable properties
57229f6
@moufmouf
More robust headers implementation.
6bdc361 
Headers now MUST be strings (no null value allowed in header).
Added HOST check in Request.
@moufmouf
Upgrading to PHPStan level 4.
c63ea35
@moufmouf
Adding casts for parameters (int to bool...)
e9eff96 
Reaching PHPStan level 5.
@moufmouf
Merge branch 'http_error_issue' into modernize_phpversion
4d22f65
@moufmouf
Adding additional check to be sure the cassette was successfully opened.
1f788e8
@moufmouf
Asserting that curl_init did succeed
cede95e
@moufmouf
Reaching PHPStan level 6
8e6530a 
Tweaked some PHPDoc statements.
Added some ignore in PHPStan where assertions are already ensuring safety.
higidi and others added 30 commits December 19, 2022 18:25
@higidi
Merge pull request php-vcr#379 from php-vcr/record_identical_requests
3e8b2b9 
Record identical requests
@oranges13
Add CURLINFO_APPCONNECT_TIME to CurlHelper
1d8349f 
Fixes php-vcr#309
@nicodemuz
Merge branch 'master' into patch-2
6585459
@higidi
Merge pull request php-vcr#361 from nicodemuz/patch-2
a8eda85 
Support for Symfony 6
@oranges13
Fix CurlHookTest.php
9bfb45b
@higidi
FIX: Read the whole stream and perform code transformations
1917d66 
First read the whole stream and afterwards perform code transformations.
This will ensure to also replace code, which is divided into chunks while
reading a file.

Fixes php-vcr#268
@higidi
FIX: Codestyle
c1f3c48
@higidi
FIX: Codestyle
9871fec
@higidi
Merge pull request php-vcr#382 from php-vcr/268-abstract-code-transfo…
bddc1fe 
…rm-splits-in-middle-of-file

FIX: Read the whole stream and perform code transformations
@higidi
Merge pull request php-vcr#381 from oranges13/add-CURLINFO_APPCONNECT…
76bcd98 
…_TIME-to-Curlhelper

Add CURLINFO_APPCONNECT_TIME to CurlHelper
@higidi
TASK: Update README.md
6d72376
@higidi
TASK: Drop support for php 8.2
3971d02 
Currently, the php-vcr library does not work well with php 8.2
so we drop the support for now until it is fixed.
@higidi
Merge pull request php-vcr#386 from php-vcr/update_readme
f7db27d 
TASK: Update README.md
@higidi
Merge pull request php-vcr#387 from php-vcr/drop_support_for_php82
5c9417b 
TASK: Drop support for php 8.2
Merge
53dfea3
Manually apply php-vcr#364
52a188a
@higidi
TASK: Apply codestyle
79a1d86
@higidi
FEATURE: Fix/review phpstan errors
6e1b61c
@higidi
Merge pull request php-vcr#393 from php-vcr/392-apply-changed-codesty…
e7f32c4 
…le-review-found-phpstan-recommendationsissues

392 apply changed codestyle review found phpstan recommendationsissues
@higidi
FEATURE: Custom workspace docker file for php 8.2 added
3bb6f71
@higidi
FEATURE: Added php 8.2 to GitHub workflow
631ee45
@higidi
TASK: Add support for php 8.2
10293cc
@higidi
TASK: Added support for php greater than 8.2.9
c9f681b
@higidi
Merge pull request php-vcr#377 from php-vcr/php_82
376dd9c 
PHP 8.2 compatibility
@higidi
FIX: Initialized typed property
7bc77ca
@higidi
Merge pull request php-vcr#396 from php-vcr/395-error-typed-property-…
7debb44 
…vcrresponse$httpversion-must-not-be-accessed-before-initialization

FIX: Initialized typed property
@Tjeerd
Merge branch 'php-vcr:master' into php-vcr-master
c863b6d
@Tjeerd
Fixes
48b5d92
Type fix
9b803cf
Support SF 7
617b688
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.