Skip to content

Merge 4.5 into develop #8720

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

Merged
merged 1,170 commits into from
Apr 7, 2024
Merged

Merge 4.5 into develop #8720

merged 1,170 commits into from
Apr 7, 2024

Conversation

kenjis
Copy link
Member

@kenjis kenjis commented Apr 7, 2024

No description provided.

 Conflicts:
	system/Session/Handlers/RedisHandler.php
like `tcp://localhost:6379?auth[user]=redis-admin&auth[pass]=admin-password`
and Unix domain socket like `unix:///tmp/redis.sock`.

 Conflicts:
	system/Session/Handlers/RedisHandler.php
We should use $options['component'] because it takes precedence.
See the code below to create a shared instance.
feat: add spark command to check php.ini
 Conflicts:
	system/Test/TestResponse.php
refactor: remove deprecated BaseService::discoverServices()
 Conflicts:
	tests/_support/Log/Handlers/TestHandler.php
kenjis added 28 commits April 6, 2024 09:24
The parent method return type is not never.
docs: update "Security Guidelines" for APIs
refactor: upgrade to PHP 8.1 with rector
vendor/bin/phpunit --migrate-configuration -c phpunit.xml.dist
Message:  file_put_contents(/home/runner/work/CodeIgniter4/CodeIgniter4/build/coverage/text/coverage.txt): Failed to open stream: No such file or directory
Location: /home/runner/work/CodeIgniter4/CodeIgniter4/vendor/phpunit/phpunit/src/Runner/CodeCoverage.php:317

Script done.
Error: Process completed with exit code 255.
1) CodeIgniter\Database\DatabaseTestCase\DatabaseTestCaseMigrationOnce1Test::testMigrationDone
TypeError: json_encode(): Argument #2 ($flags) must be of type int, bool given

/home/runner/work/CodeIgniter4/CodeIgniter4/system/Test/Constraints/SeeInDatabase.php:118
/home/runner/work/CodeIgniter4/CodeIgniter4/system/Test/DatabaseTestTrait.php:282
/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Database/DatabaseTestCase/DatabaseTestCaseMigrationOnce1Test.php:86
There was 1 error:

 1) CodeIgniter\Security\SecurityCSRFSessionRandomizeTokenTest::testCSRFVerifyPostInvalidToken
PHPUnit\Framework\Exception: PHP Warning:  hex2bin(): Hexadecimal input string must have an even length in /home/runner/work/CodeIgniter4/CodeIgniter4/system/Security/Security.php on line 378
PHP Warning:  hex2bin(): Hexadecimal input string must have an even length in /home/runner/work/CodeIgniter4/CodeIgniter4/system/Security/Security.php on line 378
There was 1 error:

1) CodeIgniter\CommonFunctionsTest::testTraceWithCSP
Test was run in child process and ended unexpectedly
@kenjis kenjis merged commit 7188805 into develop Apr 7, 2024
@kenjis kenjis deleted the 4.5 branch April 7, 2024 04: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.

3 participants