Skip to content

Commit 8a5972f

Browse files
GirgiasGirgias
authored
ext/standard: Deprecate $http_response_header (php#19464)
RFC: https://wiki.php.net/rfc/deprecations_php_8_5#deprecate_the_http_response_header_predefined_variable
1 parent e877676 commit 8a5972f

27 files changed

+189
-27
lines changed

‎Zend/zend_compile.c‎

Lines changed: 28 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -339,6 +339,7 @@ void zend_oparray_context_begin(zend_oparray_context *prev_context, zend_op_arra
339339
CG(context).try_catch_offset = -1;
340340
CG(context).current_brk_cont = -1;
341341
CG(context).last_brk_cont = 0;
342+
CG(context).has_assigned_to_http_response_header = false;
342343
CG(context).brk_cont_array = NULL;
343344
CG(context).labels = NULL;
344345
CG(context).in_jmp_frameless_branch = false;
@@ -2867,7 +2868,7 @@ static void zend_compile_class_ref(znode *result, zend_ast *name_ast, uint32_t f
28672868
}
28682869
/* }}} */
28692870

2870-
static zend_result zend_try_compile_cv(znode *result, zend_ast *ast) /* {{{ */
2871+
static zend_result zend_try_compile_cv(znode *result, zend_ast *ast, uint32_t type) /* {{{ */
28712872
{
28722873
zend_ast *name_ast = ast->child[0];
28732874
if (name_ast->kind == ZEND_AST_ZVAL) {
@@ -2884,6 +2885,16 @@ static zend_result zend_try_compile_cv(znode *result, zend_ast *ast) /* {{{ */
28842885
return FAILURE;
28852886
}
28862887

2888+
if (!CG(context).has_assigned_to_http_response_header && zend_string_equals_literal(name, "http_response_header")) {
2889+
if (type == BP_VAR_R) {
2890+
zend_error(E_DEPRECATED,
2891+
"The predefined locally scoped $http_response_header variable is deprecated,"
2892+
" call http_get_last_response_headers() instead");
2893+
} else if (type == BP_VAR_W) {
2894+
CG(context).has_assigned_to_http_response_header = true;
2895+
}
2896+
}
2897+
28872898
result->op_type = IS_CV;
28882899
result->u.op.var = lookup_cv(name);
28892900

@@ -2920,6 +2931,14 @@ static zend_op *zend_compile_simple_var_no_cv(znode *result, zend_ast *ast, uint
29202931

29212932
opline->extended_value = ZEND_FETCH_GLOBAL;
29222933
} else {
2934+
// TODO: Have a test case for this?
2935+
if (name_node.op_type == IS_CONST
2936+
&& type == BP_VAR_R
2937+
&& zend_string_equals_literal(Z_STR(name_node.u.constant), "http_response_header")) {
2938+
zend_error(E_DEPRECATED,
2939+
"The predefined locally scoped $http_response_header variable is deprecated,"
2940+
" call http_get_last_response_headers() instead");
2941+
}
29232942
opline->extended_value = ZEND_FETCH_LOCAL;
29242943
}
29252944

@@ -2991,7 +3010,7 @@ static zend_op *zend_compile_simple_var(znode *result, zend_ast *ast, uint32_t t
29913010
result->op_type = IS_TMP_VAR;
29923011
}
29933012
return opline;
2994-
} else if (zend_try_compile_cv(result, ast) == FAILURE) {
3013+
} else if (zend_try_compile_cv(result, ast, type) == FAILURE) {
29953014
return zend_compile_simple_var_no_cv(result, ast, type, delayed);
29963015
}
29973016
return NULL;
@@ -3417,7 +3436,7 @@ static void zend_compile_expr_with_potential_assign_to_self(
34173436
/* $a[0] = $a should evaluate the right $a first */
34183437
znode cv_node;
34193438

3420-
if (zend_try_compile_cv(&cv_node, expr_ast) == FAILURE) {
3439+
if (zend_try_compile_cv(&cv_node, expr_ast, BP_VAR_R) == FAILURE) {
34213440
zend_compile_simple_var_no_cv(expr_node, expr_ast, BP_VAR_R, 0);
34223441
} else {
34233442
zend_emit_op_tmp(expr_node, ZEND_QM_ASSIGN, &cv_node, NULL);
@@ -3507,7 +3526,7 @@ static void zend_compile_assign(znode *result, zend_ast *ast) /* {{{ */
35073526
/* list($a, $b) = $a should evaluate the right $a first */
35083527
znode cv_node;
35093528

3510-
if (zend_try_compile_cv(&cv_node, expr_ast) == FAILURE) {
3529+
if (zend_try_compile_cv(&cv_node, expr_ast, BP_VAR_R) == FAILURE) {
35113530
zend_compile_simple_var_no_cv(&expr_node, expr_ast, BP_VAR_R, 0);
35123531
} else {
35133532
zend_emit_op_tmp(&expr_node, ZEND_QM_ASSIGN, &cv_node, NULL);
@@ -3822,7 +3841,7 @@ static uint32_t zend_compile_args(
38223841
opcode = ZEND_SEND_VAR_EX;
38233842
CG(active_op_array)->fn_flags |= ZEND_ACC_USES_THIS;
38243843
break;
3825-
} else if (zend_try_compile_cv(&arg_node, arg) == SUCCESS) {
3844+
} else if (zend_try_compile_cv(&arg_node, arg, BP_VAR_R) == SUCCESS) {
38263845
opcode = ZEND_SEND_VAR_EX;
38273846
break;
38283847
}
@@ -5427,7 +5446,7 @@ static void zend_compile_global_var(zend_ast *ast) /* {{{ */
54275446
// TODO(GLOBALS) Forbid "global $GLOBALS"?
54285447
if (is_this_fetch(var_ast)) {
54295448
zend_error_noreturn(E_COMPILE_ERROR, "Cannot use $this as global variable");
5430-
} else if (zend_try_compile_cv(&result, var_ast) == SUCCESS) {
5449+
} else if (zend_try_compile_cv(&result, var_ast, BP_VAR_R) == SUCCESS) {
54315450
zend_op *opline = zend_emit_op(NULL, ZEND_BIND_GLOBAL, &result, &name_node);
54325451
opline->extended_value = zend_alloc_cache_slot();
54335452
} else {
@@ -5553,7 +5572,7 @@ static void zend_compile_unset(zend_ast *ast) /* {{{ */
55535572
case ZEND_AST_VAR:
55545573
if (is_this_fetch(var_ast)) {
55555574
zend_error_noreturn(E_COMPILE_ERROR, "Cannot unset $this");
5556-
} else if (zend_try_compile_cv(&var_node, var_ast) == SUCCESS) {
5575+
} else if (zend_try_compile_cv(&var_node, var_ast, BP_VAR_UNSET) == SUCCESS) {
55575576
opline = zend_emit_op(NULL, ZEND_UNSET_CV, &var_node, NULL);
55585577
} else {
55595578
opline = zend_compile_simple_var_no_cv(NULL, var_ast, BP_VAR_UNSET, 0);
@@ -6116,7 +6135,7 @@ static void zend_compile_foreach(zend_ast *ast) /* {{{ */
61166135
if (is_this_fetch(value_ast)) {
61176136
zend_error_noreturn(E_COMPILE_ERROR, "Cannot re-assign $this");
61186137
} else if (value_ast->kind == ZEND_AST_VAR &&
6119-
zend_try_compile_cv(&value_node, value_ast) == SUCCESS) {
6138+
zend_try_compile_cv(&value_node, value_ast, BP_VAR_R) == SUCCESS) {
61206139
SET_NODE(opline->op2, &value_node);
61216140
} else {
61226141
opline->op2_type = IS_VAR;
@@ -10842,7 +10861,7 @@ static void zend_compile_isset_or_empty(znode *result, zend_ast *ast) /* {{{ */
1084210861
if (is_this_fetch(var_ast)) {
1084310862
opline = zend_emit_op(result, ZEND_ISSET_ISEMPTY_THIS, NULL, NULL);
1084410863
CG(active_op_array)->fn_flags |= ZEND_ACC_USES_THIS;
10845-
} else if (zend_try_compile_cv(&var_node, var_ast) == SUCCESS) {
10864+
} else if (zend_try_compile_cv(&var_node, var_ast, BP_VAR_IS) == SUCCESS) {
1084610865
opline = zend_emit_op(result, ZEND_ISSET_ISEMPTY_CV, &var_node, NULL);
1084710866
} else {
1084810867
opline = zend_compile_simple_var_no_cv(result, var_ast, BP_VAR_IS, 0);

‎Zend/zend_compile.h‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -208,6 +208,7 @@ typedef struct _zend_oparray_context {
208208
zend_string *active_property_info_name;
209209
zend_property_hook_kind active_property_hook_kind;
210210
bool in_jmp_frameless_branch;
211+
bool has_assigned_to_http_response_header;
211212
} zend_oparray_context;
212213

213214
/* Class, property and method flags class|meth.|prop.|const*/

‎ext/standard/tests/http/bug75535.phpt‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ $responses = array(
1515
['pid' => $pid, 'uri' => $uri] = http_server($responses, $output);
1616

1717
var_dump(file_get_contents($uri));
18-
var_dump($http_response_header);
18+
var_dump(http_get_last_response_headers());
1919

2020
http_server_kill($pid);
2121

‎ext/standard/tests/http/bug80838.phpt‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,8 @@ var_dump(http_get_last_response_headers());
3333
http_server_kill($pid);
3434

3535
?>
36-
--EXPECT--
36+
--EXPECTF--
37+
Deprecated: The predefined locally scoped $http_response_header variable is deprecated, call http_get_last_response_headers() instead in %s on line 23
3738
NULL
3839
array(3) {
3940
[0]=>

‎ext/standard/tests/http/gh9316.phpt‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,8 @@ for ($i = 0; $i < count($responses); ++$i) {
3030
http_server_kill($pid);
3131

3232
?>
33-
--EXPECT--
33+
--EXPECTF--
34+
Deprecated: The predefined locally scoped $http_response_header variable is deprecated, call http_get_last_response_headers() instead in %s on line 17
3435
http_get_last_response_headers() before stream layer call:
3536
NULL
3637
$http_response_header

‎ext/standard/tests/http/ghsa-52jp-hrpf-2jff-001.phpt‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ $clientCode = <<<'CODE'
3636
$ctx = stream_context_create();
3737
stream_context_set_params($ctx, array("notification" => "stream_notification_callback"));
3838
var_dump(trim(file_get_contents("http://{{ ADDR }}", false, $ctx)));
39-
var_dump($http_response_header);
39+
var_dump(http_get_last_response_headers());
4040
CODE;
4141

4242
include sprintf("%s/../../../openssl/tests/ServerClientTestCase.inc", __DIR__);

‎ext/standard/tests/http/ghsa-52jp-hrpf-2jff-002.phpt‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ $clientCode = <<<'CODE'
3636
$ctx = stream_context_create();
3737
stream_context_set_params($ctx, array("notification" => "stream_notification_callback"));
3838
var_dump(trim(file_get_contents("http://{{ ADDR }}", false, $ctx)));
39-
var_dump($http_response_header);
39+
var_dump(http_get_last_response_headers());
4040
CODE;
4141

4242
include sprintf("%s/../../../openssl/tests/ServerClientTestCase.inc", __DIR__);

‎ext/standard/tests/http/ghsa-hgf5-96fm-v528-001.phpt‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ $clientCode = <<<'CODE'
3333
];
3434
$ctx = stream_context_create($opts);
3535
var_dump(explode("\r\n", base64_decode(file_get_contents("http://user:pwd@{{ ADDR }}", false, $ctx))));
36-
var_dump($http_response_header);
36+
var_dump(http_get_last_response_headers());
3737
CODE;
3838

3939
include sprintf("%s/../../../openssl/tests/ServerClientTestCase.inc", __DIR__);

‎ext/standard/tests/http/ghsa-hgf5-96fm-v528-002.phpt‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ $clientCode = <<<'CODE'
3333
];
3434
$ctx = stream_context_create($opts);
3535
var_dump(explode("\r\n", base64_decode(file_get_contents("http://user:pwd@{{ ADDR }}", false, $ctx))));
36-
var_dump($http_response_header);
36+
var_dump(http_get_last_response_headers());
3737
CODE;
3838

3939
include sprintf("%s/../../../openssl/tests/ServerClientTestCase.inc", __DIR__);

‎ext/standard/tests/http/ghsa-hgf5-96fm-v528-003.phpt‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ $clientCode = <<<'CODE'
3333
];
3434
$ctx = stream_context_create($opts);
3535
var_dump(explode("\r\n", base64_decode(file_get_contents("http://user:pwd@{{ ADDR }}", false, $ctx))));
36-
var_dump($http_response_header);
36+
var_dump(http_get_last_response_headers());
3737
CODE;
3838

3939
include sprintf("%s/../../../openssl/tests/ServerClientTestCase.inc", __DIR__);

0 commit comments

Comments
 (0)