Skip to content

Deprecate returning non-string values from a user output handler #18932

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

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

Deprecate returning non-string values from a user output handler #18932

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
f4c3fde
Deprecate returning non-string values from a user output handler
DanielEScherzer Jun 24, 2025
340c7fd
Only disable the problematic output handler
DanielEScherzer Jun 24, 2025
720923e
Tests
DanielEScherzer Jun 24, 2025
cfe4d71
upgrading
DanielEScherzer Jun 24, 2025
51892ba
Handlers might be freed from OOM
DanielEScherzer Jun 26, 2025
6a2d6c1
Location not always given
DanielEScherzer Jun 26, 2025
ab5acde
context is also unavailable
DanielEScherzer Jun 26, 2025
25c35b8
No tabs
DanielEScherzer Jun 26, 2025
8144c55
Memory limits
DanielEScherzer Jun 28, 2025
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Memory limits
  • Loading branch information
@DanielEScherzer
DanielEScherzer committed Jun 28, 2025
commit 8144c5512b6de10719ef34990b723012f1c27bd5
2 changes: 2 additions & 0 deletions tests/output/ob_start_callback_bad_return/handler_false_removed.phpt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
--TEST--
ob_start(): Check behaviour with deprecation when OOM triggers handler removal (handler returns false)
--INI--
memory_limit=2M
--FILE--
<?php

Expand Down
2 changes: 2 additions & 0 deletions tests/output/ob_start_callback_bad_return/handler_true_removed.phpt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
--TEST--
ob_start(): Check behaviour with deprecation when OOM triggers handler removal (handler returns true)
--INI--
memory_limit=2M
--FILE--
<?php

Expand Down
2 changes: 2 additions & 0 deletions tests/output/ob_start_callback_bad_return/handler_zero_removed.phpt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
--TEST--
ob_start(): Check behaviour with deprecation when OOM triggers handler removal (handler returns zero)
--INI--
memory_limit=2M
--FILE--
<?php

Expand Down