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.

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

added PHP version check, fixed #19925 #19936

Merged
merged 9 commits into from
Sep 4, 2023
Merged

added PHP version check, fixed #19925 #19936

merged 9 commits into from
Sep 4, 2023

Conversation

schmunk42
Copy link
Contributor

@schmunk42 schmunk42 commented Aug 24, 2023

Q A
Is bugfix? ✔️
New feature?
Breaks BC?
Fixed issues #19925

fixes Docker build yiisoft/yii2-docker#168

builds here are also green

@what-the-diff
Copy link

what-the-diff bot commented Aug 24, 2023

PR Summary

  • Patch to address libmagic anomaly in specific PHP versions
    In the files mimeTypes.php, MimeTest.php, and FileValidatorTest.php, we have made improvements to handle a known issue with the libmagic library, which is used for identifying file types. The patch is triggered when the PHP version in effect ranges from 8.1.0 to 8.1.21 or from 8.2.0 to 8.2.8. This should offer better performance and increased stability in these PHP environments.

@codecov
Copy link

codecov bot commented Aug 24, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (4ffb58e) 48.95% compared to head (9813624) 48.95%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #19936   +/-   ##
=======================================
  Coverage   48.95%   48.95%           
=======================================
  Files         445      445           
  Lines       42807    42807           
=======================================
  Hits        20957    20957           
  Misses      21850    21850           
Files Changed Coverage Δ
framework/helpers/mimeTypes.php 0.30% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@samdark
Copy link
Member

samdark commented Aug 26, 2023

@schmunk42 would you please add a line for CHANGELOG? Thanks.

@schmunk42
Copy link
Contributor Author

Changelog added, conflicts resolved.
Would be cool if this could get merged since it's blocking Docker builds.

@bizley bizley merged commit e8c4a7e into master Sep 4, 2023
72 of 94 checks passed
@bizley bizley deleted the feature/fix-19925 branch September 4, 2023 08:09
@bizley
Copy link
Member

bizley commented Sep 4, 2023

👍🏻

rob006 added a commit to rob006/yii2-dev that referenced this pull request Oct 6, 2023
yii-bot pushed a commit to yiisoft/yii2-framework that referenced this pull request Oct 6, 2023
samdark added a commit that referenced this pull request Oct 19, 2023
* added pcntl to requirements check

* Fix #13920: Fixed erroneous validation for specific cases

* Fix #13920: Added my name to CHANGELOG.md

* trim(): Passing null to parameter #1 ($string) of type string is deprecated

* Added section about Unsafe Reflection in Security best practices doc (#19948)

Co-authored-by: Bizley <[email protected]>

* Update CHANGELOG.md

* Fixed tests.

* Fix #13920: Add unit test

* Update `bower-asset/inputmask`, `bower-asset/punycode`.

* added CHANGELOG line

* Fix order.

* fix: #19978 - Mistake in Korean translation

* fix: keep doublequote

* Do not duplicate log messages in memory

* Update framework/log/FileTarget.php

Co-authored-by: Bizley <[email protected]>

* Update concept-di-container.md

Сслка "Конфигурация приложения" была не верной, она вела на страницу "Service-locator'a"

* Update CHANGELOG.md

* release version 2.0.49.1

* prepare for next release

* Revert changes in `mimeTypes.php` from 4a1f2c6

restores #19936

* update actions/checkout to v4

* Fix mime type generator

* Added note

* Update structure-controllers.md (#20003)

добавил пропущенное слово "как"

* Update CHANGELOG.md

2.0.49.2 changelog

* release version 2.0.49.2

* prepare for next release

* New methods: BaseActiveRecord::loadRelations() and BaseActiveRecord::loadRelationsFor().

* Fixed a bug where the yii serve command would break if a custom router was supplied and it had a space in the path

* Fix `MaskedInputAsset::class`.

* Fix #19927: Fixed `console\controllers\MessageController` when saving translations to database: fixed FK error when adding new string and language at the same time, checking/regenerating all missing messages and dropping messages for unused languages

* Added 'zh' into 'framework/messages/config.php' (#19995)

---------

Co-authored-by: Tobias Munk <[email protected]>
Co-authored-by: Tim Fischer <[email protected]>
Co-authored-by: Tim Fischer <[email protected]>
Co-authored-by: Alexander Makarov <[email protected]>
Co-authored-by: Oleg Poludnenko <[email protected]>
Co-authored-by: Bizley <[email protected]>
Co-authored-by: Sonia Zorba <[email protected]>
Co-authored-by: Tobias Munk <[email protected]>
Co-authored-by: Akbar Herlambang <[email protected]>
Co-authored-by: lubosdz <[email protected]>
Co-authored-by: Yuriy Bachevskiy <[email protected]>
Co-authored-by: Robert Korulczyk <[email protected]>
Co-authored-by: salehhashemi1992 <[email protected]>
Co-authored-by: PowerGamer1 <[email protected]>
Co-authored-by: Brad Bell <[email protected]>
Co-authored-by: Alexandru Trandafir Catalin <[email protected]>
Co-authored-by: Nabi KaramAliZadeh <[email protected]>
yii-bot pushed a commit to yiisoft/yii2-framework that referenced this pull request Oct 19, 2023
* added pcntl to requirements check

* Fix #13920: Fixed erroneous validation for specific cases

* Fix #13920: Added my name to CHANGELOG.md

* trim(): Passing null to parameter #1 ($string) of type string is deprecated

* Added section about Unsafe Reflection in Security best practices doc (#19948)

Co-authored-by: Bizley <[email protected]>

* Update CHANGELOG.md

* Fixed tests.

* Fix #13920: Add unit test

* Update `bower-asset/inputmask`, `bower-asset/punycode`.

* added CHANGELOG line

* Fix order.

* fix: #19978 - Mistake in Korean translation

* fix: keep doublequote

* Do not duplicate log messages in memory

* Update framework/log/FileTarget.php

Co-authored-by: Bizley <[email protected]>

* Update concept-di-container.md

Сслка "Конфигурация приложения" была не верной, она вела на страницу "Service-locator'a"

* Update CHANGELOG.md

* release version 2.0.49.1

* prepare for next release

* Revert changes in `mimeTypes.php` from 4a1f2c6b9bc90427e91da73f5e8c8fa33d3c53c1

restores yiisoft/yii2#19936

* update actions/checkout to v4

* Fix mime type generator

* Added note

* Update structure-controllers.md (#20003)

добавил пропущенное слово "как"

* Update CHANGELOG.md

2.0.49.2 changelog

* release version 2.0.49.2

* prepare for next release

* New methods: BaseActiveRecord::loadRelations() and BaseActiveRecord::loadRelationsFor().

* Fixed a bug where the yii serve command would break if a custom router was supplied and it had a space in the path

* Fix `MaskedInputAsset::class`.

* Fix #19927: Fixed `console\controllers\MessageController` when saving translations to database: fixed FK error when adding new string and language at the same time, checking/regenerating all missing messages and dropping messages for unused languages

* Added 'zh' into 'framework/messages/config.php' (#19995)

---------

Co-authored-by: Tobias Munk <[email protected]>
Co-authored-by: Tim Fischer <[email protected]>
Co-authored-by: Tim Fischer <[email protected]>
Co-authored-by: Alexander Makarov <[email protected]>
Co-authored-by: Oleg Poludnenko <[email protected]>
Co-authored-by: Bizley <[email protected]>
Co-authored-by: Sonia Zorba <[email protected]>
Co-authored-by: Tobias Munk <[email protected]>
Co-authored-by: Akbar Herlambang <[email protected]>
Co-authored-by: lubosdz <[email protected]>
Co-authored-by: Yuriy Bachevskiy <[email protected]>
Co-authored-by: Robert Korulczyk <[email protected]>
Co-authored-by: salehhashemi1992 <[email protected]>
Co-authored-by: PowerGamer1 <[email protected]>
Co-authored-by: Brad Bell <[email protected]>
Co-authored-by: Alexandru Trandafir Catalin <[email protected]>
Co-authored-by: Nabi KaramAliZadeh <[email protected]>
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.

5 participants