mirror of
https://github.com/apache/thrift.git
synced 2026-10-29 00:46:21 +08:00
Volodymyr Panivko
f69b418f1f
Client: php PHP was the only major-language entry stuck on buffered/framed sockets while Python, Go, Node, C++, Java, Lua, Dart, JS, D, and Haskell already declared http in their cross-test transports. Server (test/php/TestServer.php + HttpRouter.php + HttpServer.php): - On --transport=http, TestServer pcntl_exec()s into 'php -S' with HttpRouter.php as the router. The chameleon HttpRouter dispatches on PHP_SAPI: cli launches php -S with itself as the per-request handler; cli-server invokes HttpServer. - HttpServer reads php://input via TMemoryBuffer, peeks the message type, and for ONEWAY sends an empty HTTP 200 immediately while forking the handler so strict clients (Java) measure fast oneway latency. Sync path echoes the processor's response bytes. - protocols.php centralises the protocol-name -> TProtocolFactory dispatch, shared between the socket entry point and the cli-server router. Client (test/php/TestClient.php): - Adds an http branch that uses TPsrHttpClient (THRIFT-6010) against http://127.0.0.1:$port/. Drops the duplicate makeProtocol() helper in favour of thrift_test_protocol_factory()->getProtocol(), removes the unused $host argv parsing, and normalises the file to PSR-12. composer.json: adds guzzlehttp/guzzle ^7.8 to require-dev so PSR-18 discovery resolves a real client at cross-test time. test/tests.json: adds 'http' to PHP server and client transports. test/known_failures_Linux.json: lists php-cpp_*_http-ip (cpp THttpClient does not handle Connection: close between calls — tracked as THRIFT-6060) and nodejs-php_*_http-ip (Node http server mounts /test, not /, matching the existing cpp-/d-/go-nodejs known failures). Generated-by: Claude Opus 4.7
78 lines
2.7 KiB
PHP
78 lines
2.7 KiB
PHP
<?php
|
|
|
|
/*
|
|
* Licensed to the Apache Software Foundation (ASF) under one
|
|
* or more contributor license agreements. See the NOTICE file
|
|
* distributed with this work for additional information
|
|
* regarding copyright ownership. The ASF licenses this file
|
|
* to you under the Apache License, Version 2.0 (the
|
|
* "License"); you may not use this file except in compliance
|
|
* with the License. You may obtain a copy of the License at
|
|
*
|
|
* http://www.apache.org/licenses/LICENSE-2.0
|
|
*
|
|
* Unless required by applicable law or agreed to in writing,
|
|
* software distributed under the License is distributed on an
|
|
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
|
|
* KIND, either express or implied. See the License for the
|
|
* specific language governing permissions and limitations
|
|
* under the License.
|
|
*/
|
|
|
|
/**
|
|
* Two roles, dispatched on PHP_SAPI:
|
|
*
|
|
* - CLI (sourced via require from TestServer.php): `pcntl_exec` into
|
|
* `php -S` with this file as the per-request router. $port and $protocol
|
|
* are read from the caller's scope.
|
|
* - cli-server: handle one HTTP request via HttpServer. The protocol is
|
|
* forwarded from the launcher through the THRIFT_TEST_PROTOCOL env var.
|
|
*/
|
|
|
|
// cli-server: per-request handler.
|
|
if (PHP_SAPI === 'cli-server') {
|
|
require_once __DIR__ . '/../../vendor/autoload.php';
|
|
require_once __DIR__ . '/protocols.php';
|
|
require_once __DIR__ . '/HttpServer.php';
|
|
|
|
// ThriftTest generated classes are not in Composer's PSR-4 map; register
|
|
// the classmap loader before requiring Handler.php so its
|
|
// `implements ThriftTestIf` resolves correctly.
|
|
$loader = new \Thrift\ClassLoader\ThriftClassLoader();
|
|
$loader->registerDefinition('ThriftTest', __DIR__ . '/gen-php-classmap');
|
|
$loader->register();
|
|
|
|
require_once __DIR__ . '/Handler.php';
|
|
|
|
$protocolFactory = thrift_test_protocol_factory(getenv('THRIFT_TEST_PROTOCOL') ?: 'binary');
|
|
$processor = new ThriftTest\ThriftTestProcessor(new Handler());
|
|
|
|
(new HttpServer($processor, $protocolFactory))->serve();
|
|
return;
|
|
}
|
|
|
|
// CLI launcher: sourced from TestServer.php; $port and $protocol are in scope.
|
|
if (!function_exists('pcntl_exec')) {
|
|
fwrite(STDERR, "PHP HTTP cross-test requires ext-pcntl.\n");
|
|
exit(1);
|
|
}
|
|
|
|
$env = getenv();
|
|
$env['THRIFT_TEST_PROTOCOL'] = $protocol;
|
|
|
|
echo "Starting the Test server (HTTP via php -S)...\n";
|
|
|
|
// -d flags mirror the launcher invocation in test/tests.json — keep in sync.
|
|
pcntl_exec(PHP_BINARY, [
|
|
'-dextension_dir=php_ext_dir',
|
|
'-dextension=thrift_protocol.so',
|
|
'-ddisplay_errors=stderr',
|
|
'-dlog_errors=0',
|
|
'-derror_reporting=E_ALL',
|
|
'-S', '127.0.0.1:' . $port,
|
|
__FILE__,
|
|
], $env);
|
|
|
|
fwrite(STDERR, "pcntl_exec failed\n");
|
|
exit(1);
|