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

Release 11.5 #1007

Merged
merged 9 commits into from
Jan 9, 2025
Merged

Release 11.5 #1007

merged 9 commits into from
Jan 9, 2025

Conversation

hahuja2
Copy link
Contributor

@hahuja2 hahuja2 commented Jan 9, 2025

No description provided.

lavarou and others added 9 commits December 9, 2024 14:38
Fix QOL aliases and allow multiple devenvs to run simultanously.
Address CVE-2024-45338 by upgrading golang.org/x/net to v0.33.0

go get golang.org/x/[email protected]
go: upgraded golang.org/x/net v0.23.0 => v0.33.0
go: upgraded golang.org/x/sys v0.18.0 => v0.28.0
go: upgraded golang.org/x/text v0.14.0 => v0.21.0
Concerns about future/past proofing to the checking prioritized the
non-eval implementation vs using the eval method for checking the
version. However, aws-sdk-php internal functions frequently change
without notice in the release notes. An extra eval is used to avoid the
fatal error that would occur in the VERY unlikely event of Aws/Sdk.php
not being found and which addresses the previous concerns while
decoupling versioning from internal functions.
Fix agent build scan by pinning runner to ubuntu-22.04. ubuntu-24.04,
which is what ubuntu-latest has recently become, doesn't have agent's
build dependency (libpcre3-dev) installed.
`codeql-action/upload-sarif` needs to know where the scanned code
was checked out if it has been checked out to a subdir of `github.workspace`.
Bumps monolog and guzzle versions for 8.4 support from those libraries.
monolog2 -> 2.10
monolog3 -> 3.8
guzzle -> 7.9.2

Adds 8.4 to builds/tests/installers. Requires some updates to our
make/configure scripts to adapt to changes in PHP autoconf macros.

Adjusts tests to account for new closure naming in PHP 8.4

Adjusts tests to account for deprecation of E_USER_ERROR

---------

Co-authored-by: James <[email protected]>
Co-authored-by: Michal Nowacki <[email protected]>
@hahuja2 hahuja2 requested a review from bduranleau-nr January 9, 2025 20:37
@newrelic-php-agent-bot
Copy link

newrelic-php-agent-bot commented Jan 9, 2025

Test Suite Status Result
Multiverse 8/8 passing
SOAK 72/72 passing

@codecov-commenter
Copy link

codecov-commenter commented Jan 9, 2025

Codecov Report

Attention: Patch coverage is 49.12281% with 29 lines in your changes missing coverage. Please review.

Project coverage is 78.04%. Comparing base (eab6b4b) to head (8a73dde).

Files with missing lines Patch % Lines
agent/fw_drupal8.c 6.89% 27 Missing ⚠️
agent/lib_aws_sdk_php.c 71.42% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1007      +/-   ##
==========================================
- Coverage   78.68%   78.04%   -0.64%     
==========================================
  Files         196      196              
  Lines       27047    27105      +58     
==========================================
- Hits        21281    21155     -126     
- Misses       5766     5950     +184     
Flag Coverage Δ
agent-for-php-7.2 78.04% <21.62%> (-0.66%) ⬇️
agent-for-php-7.3 78.05% <21.62%> (-0.66%) ⬇️
agent-for-php-7.4 77.76% <21.62%> (-0.66%) ⬇️
agent-for-php-8.0 77.13% <21.62%> (-1.31%) ⬇️
agent-for-php-8.1 77.79% <21.62%> (-0.63%) ⬇️
agent-for-php-8.2 77.38% <21.62%> (-0.63%) ⬇️
agent-for-php-8.3 77.38% <21.62%> (-0.63%) ⬇️
agent-for-php-8.4 77.40% <49.12%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@hahuja2 hahuja2 merged commit 5b320e8 into main Jan 9, 2025
126 of 127 checks passed
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.

7 participants