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

feat(agent): add support for PHP 8.3 #737

Closed
wants to merge 6 commits into from
Closed

feat(agent): add support for PHP 8.3 #737

wants to merge 6 commits into from

Conversation

bduranleau-nr
Copy link
Contributor

No description provided.

@lavarou lavarou changed the title feat: PHP 8.3 feat(agent): add support for PHP 8.3 Sep 29, 2023
@codecov-commenter
Copy link

codecov-commenter commented Oct 2, 2023

Codecov Report

Merging #737 (24457e0) into oapi (2b9e3b4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             oapi     #737   +/-   ##
=======================================
  Coverage   78.59%   78.59%           
=======================================
  Files         191      191           
  Lines       26627    26627           
=======================================
  Hits        20928    20928           
  Misses       5699     5699           
Flag Coverage Δ
agent-for-php-7.0 77.37% <ø> (ø)
agent-for-php-7.1 77.09% <ø> (ø)
agent-for-php-7.2 77.60% <ø> (ø)
agent-for-php-7.3 77.62% <ø> (ø)
agent-for-php-7.4 77.36% <ø> (ø)
agent-for-php-8.0 77.16% <ø> (ø)
agent-for-php-8.1 77.11% <ø> (ø)
agent-for-php-8.2 76.72% <ø> (ø)

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

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

PHP 8.3 will invoke the user_exception_handler when an exception is
encountered (if one is registered). This results in an inversion of the
expectation from pre-8.3  of failing out of the zend_try/zend_catch
block of code that sets the zend_result to FAILURE and returns NULL to
the expr variable assigned to nr_php_call().
8.3 logic allows our newrelic_exception_handler to intercept the
exception, skip the zend_catch, and return SUCCESS in
nr_php_call_user_func() which returns a non-NULL value.
@adri adri mentioned this pull request Oct 4, 2023
@rvanlaak
Copy link

rvanlaak commented Nov 7, 2023

After a great start in September, it now seems it has been a bit silent on this PR. Any update on what still would need to happen? The final release candidate will be available two days from now, and the general availability release will be in three weeks: https://wiki.php.net/todo/php83

@Ak-x
Copy link

Ak-x commented Nov 9, 2023

Hello @rvanlaak, the Php 8.3 release depends on our work in ObserverAPI. Once resolved 8.3 should be straightforward.

@rvanlaak
Copy link

Thanks for the update @Ak-x , that looks like a major change to the approach, exciting! Work on ObserverAPI started over a year ago and does not have a draft PR yet, while in the meanwhile the PHP 8.2 support on the agent was able to get released.

Can you share some more visibility on the prioritization and dependency of ObserverAPI you refer to, over agent language support? Teams are eager on that in order to being able to complete the tech roadmapping they're working on.

@parallels999
Copy link

PHP 8.3 already released

@abobija
Copy link

abobija commented Dec 6, 2023

Bump

@rvanlaak
Copy link

rvanlaak commented Dec 7, 2023

Based on the branch name of a new draft PR on adding PHP 8.3 support: #797, it seems the dependency on ObserverAPI has been dropped.

@newrelic newrelic deleted a comment from andypost Dec 7, 2023
@lavarou lavarou removed this from the PHP 8.3 Support milestone Dec 18, 2023
@lavarou lavarou added the oapi label Dec 18, 2023
@bduranleau-nr
Copy link
Contributor Author

PHP 8.3 support was released in 10.15.0.4.
OAPI will be rebased to include 8.3 support.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants