Repository navigation
Regression: curl_escape()/curl_unescape() still 1-arg — php-src requires CurlHandle (re-#13588, ext/curl/interface.c) #20493
Description
Activity
claim: PHP Compiler Ext — I/O & Crypto — starting this run
Trust probe (this workspace):
docker infoOK; vendor presentphp script/bootstrap-inventory.php --check→ OK 5646/5646./script/docker-exec.sh -- bash -lc 'make bootstrap-selfhost-link'→ OK/compiler/build/selfhost./script/release-readiness.sh --json— still running; will note if red
Implementing curl_escape/curl_unescape 2-arg CurlHandle signature (php-src-strict, re-#13588).
- added a commit that references this issue
on Jul 18, 2026 merged: #20514
Verification
php bin/vm.php test/repro/issue_20493_curl_escape_handle.php→ok 'a%20b'/ok 'a b'/1arg ArgumentCountError/badhandle TypeError:…CurlHandle- VM compliance
--filter curl_escape→ OK (4) - JIT compliance
--filter curl_escape→ OK (3) - unit CurlEscape/CurlModule/CurlExtensionPolicy → OK
- php-src-strict:
curl_escape(CurlHandle, string)/curl_unescape(CurlHandle, string)
Category
stdlib· php-src-strict · ext/curl · signature parity (re-#13588)Problem
#13588 gated
curl_escape/curl_unescapeonextension_loaded('curl'), but the arity/signature fix never landed. php-src requirescurl_escape(CurlHandle $handle, string $string); this compiler still registers a 1-arg string-only API.Verified 2026-07-18 on master (
b6b30adbf) with curl advertised (extension_loaded('curl')true).curl_escape($ch, 'a b')'a%20b'curl_escape('a b')'a%20b'(accepts phantom 1-arg)curl_unescapephp-src reference
ext/curl/interface.c—PHP_FUNCTION(curl_escape),PHP_FUNCTION(curl_unescape)ext/curl/curl.stub.php—curl_escape(CurlHandle $handle, string $string): string|falsePHP implementation target
ext/curl/curl_escape.php,ext/curl/curl_unescape.php— require CurlHandle (or curl resource) as arg0; coerce string as arg1; keep encode/decode inVmCurlEscape(PHP-in-PHP)JitUrlencodeshortcut arityruntime/*.clogicRepro
Done when
curl_escape($ch, 'a b')/curl_unescape($ch, 'a%20b')match Zend when curl is loadedcurl_escape('a b')TypeErrors like php-src (CurlHandle required).phptundertest/compliance/cases/curl/(or repro + unit)