summaryrefslogtreecommitdiff
path: root/ext/soap
Commit message (Collapse)AuthorAgeFilesLines
* Use zend_string_equals() API instead of strcmp() in SOAP extensionGeorge Peter Banyard2021-03-172-4/+4
|
* Change the order of properties used for var_dump(), serialize(), comparison, ↵Dmitry Stogov2021-03-011-1/+1
| | | | | | etc. Now properties are ordered according to their layout in zend_object structure.
* Remove useless conditional #ifdefPatrick Allaert2021-02-271-4/+1
| | | | | | Closes GH-6737 Signed-off-by: George Peter Banyard <girgias@php.net>
* Improve class entry generationMáté Kocsis2021-02-161-7/+6
| | | | Related to GH-6701
* Generate class entries for snmp, soap, sockets, sodium, sqlite3, sysv*, tidyMáté Kocsis2021-02-153-28/+70
| | | | Closes GH-6696
* Merge branch 'PHP-8.0'Nikita Popov2021-02-101-1/+1
|\ | | | | | | | | * PHP-8.0: Don't pass null action to __doRequest
| * Don't pass null action to __doRequestNikita Popov2021-02-101-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The parameter is not nullable, so it will be interpreted as an empty string anyway. The entire code here is pretty confusing though, and probably deserves a second loop. The HTTP code only send SOAPAction/action if soapaction is non-NULL -- but it always is, because it is accepted through a non-nullable string parameter. Regarding the SOAPAction header, it appears that always sending it is actually a requirement of the standard: > An HTTP client MUST use this header field when issuing a SOAP > HTTP Request. Although it does make a distinction between absence of value and an empty string: > The header field value of empty string ("") means that the intent > of the SOAP message is provided by the HTTP Request-URI. No value > means that there is no indication of the intent of the message. The empty string interpretation appears to be the desired one. However, for the action MIME tag the SOAP 1.2 Part 2 specification says that > The media type specifies an optional action parameter, which can > be used to optimize dispatch or routing, among other things. but also > The SOAP Action feature defines a single property, which is > described in Table 14. The value of this property MUST be an > absolute URI[RFC 3986] and MUST NOT be empty. which would indicate that we should not be sending an empty action here. As I'm not familiar with SOAP and this is long-standing behavior, I'm just leaving this alone for now...
* | Merge branch 'PHP-8.0'Nikita Popov2021-02-102-7/+4
|\ \ | |/ | | | | | | | | * PHP-8.0: Clarify that location is required in do_request Regenerate arginfo file
| * Clarify that location is required in do_requestNikita Popov2021-02-101-5/+2
| | | | | | | | | | | | As far as I can tell, the location is always non-null here, and the code wouldn't be able to meaningfully work without a location.
| * Regenerate arginfo fileNikita Popov2021-02-101-2/+2
| | | | | | | | Somehow missed this in the previous commit.
* | Merge branch 'PHP-8.0'Nikita Popov2021-02-102-2/+2
|\ \ | |/ | | | | | | * PHP-8.0: Accept null $location in SoapClient::__setLocation()
| * Accept null $location in SoapClient::__setLocation()Nikita Popov2021-02-102-2/+2
| | | | | | | | | | | | | | | | | | | | | | Currently an empty string is used to unset the location. Once again, it makes more sense to use a null value for this purpose (though the special behavior of empty strings is retained). The code comment above the function also explicitly indicates that null should be accepted, and the function does return null rather than an empty string for the old location value (if it is missing).
* | Merge branch 'PHP-8.0'Nikita Popov2021-02-103-7/+7
|\ \ | |/ | | | | | | * PHP-8.0: Make SoapVar arguments nullable
| * Make SoapVar arguments nullableNikita Popov2021-02-103-7/+7
| | | | | | | | | | | | | | $typeName, $typeNamespace, $nodeName and $nodeNamespace all special-case the empty string and don't set the property entirely in that case. It makes more sense to use null to indicate absence here (though of course the empty string behavior is retained).
* | Merge branch 'PHP-8.0'Nikita Popov2021-02-021-2/+1
|\ \ | |/ | | | | | | * PHP-8.0: Fix persistent leak on load_wsdl_ex failure
| * Merge branch 'PHP-7.4' into PHP-8.0Nikita Popov2021-02-021-2/+1
| |\ | | | | | | | | | | | | * PHP-7.4: Fix persistent leak on load_wsdl_ex failure
| | * Fix persistent leak on load_wsdl_ex failureNikita Popov2021-02-021-2/+1
| | | | | | | | | | | | | | | | | | Move the load_wsdl_ex call into the zend_try that destroys the docs hash table. The wsdl will be inserted into docs early on, and will thus be released on subsequent bailout.
* | | Merge branch 'PHP-8.0'Nikita Popov2021-02-011-1/+1
|\ \ \ | |/ / | | | | | | | | | | | | * PHP-8.0: Fix build crc32 mac build fix
| * | Merge branch 'PHP-7.4' into PHP-8.0Nikita Popov2021-02-011-1/+1
| |\ \ | | |/ | | | | | | | | | * PHP-7.4: Fix build
| | * Fix buildNikita Popov2021-02-011-1/+1
| | |
* | | Merge branch 'PHP-8.0'Stanislav Malyshev2021-02-014-14/+37
|\ \ \ | |/ / | | | | | | | | | * PHP-8.0: Fix bug #80672 - Null Dereference in SoapClient
| * | Merge branch 'PHP-7.4' into PHP-8.0Stanislav Malyshev2021-01-314-14/+37
| |\ \ | | |/ | | | | | | | | | * PHP-7.4: Fix bug #80672 - Null Dereference in SoapClient
| | * Merge branch 'PHP-7.3' into PHP-7.4Stanislav Malyshev2021-01-314-14/+37
| | |\ | | | | | | | | | | | | | | | | * PHP-7.3: Fix bug #80672 - Null Dereference in SoapClient
| | | * Fix bug #80672 - Null Dereference in SoapClientStanislav Malyshev2021-01-314-14/+37
| | | |
* | | | Replace zend_bool uses with boolNikita Popov2021-01-158-32/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | We're starting to see a mix between uses of zend_bool and bool. Replace all usages with the standard bool type everywhere. Of course, zend_bool is retained as an alias.
* | | | Use ephemeral port in more server testsNikita Popov2020-10-232-4/+0
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Port the main php_cli_server.inc to use ephemeral ports, thus allowing CLI server tests to be parallelized. A complication here is that we also need to give each test a separate doc root, to avoid index.php files writing over each other. Closes GH-6375.
* | | Update ext/soap parameter namesNikita Popov2020-10-084-65/+45
| | | | | | | | | | | | Closes GH-6300.
* | | Accept bool in SoapClient::__doRequestNikita Popov2020-10-084-9/+9
| | | | | | | | | | | | $one_way is a boolean argument.
* | | Merge branch 'PHP-7.4'Dmitry Stogov2020-09-301-2/+0
|\ \ \ | |/ / | | | | | | | | | * PHP-7.4: Fixed incorrect error message
| * | Merge branch 'PHP-7.3' into PHP-7.4Dmitry Stogov2020-09-301-2/+0
| |\ \ | | |/ | | | | | | | | | * PHP-7.3: Fixed incorrect error message
| | * Fixed incorrect error messageGeorge Peter Banyard2020-09-301-2/+0
| | |
* | | Merge branch 'PHP-7.4'Dmitry Stogov2020-09-301-2/+2
|\ \ \ | |/ / | | | | | | | | | * PHP-7.4: Fixed incorrect logical condition
| * | Merge branch 'PHP-7.3' into PHP-7.4Dmitry Stogov2020-09-301-2/+2
| |\ \ | | |/ | | | | | | | | | * PHP-7.3: Fixed incorrect logical condition
| | * Fixed incorrect logical conditionGeorge Peter Banyard2020-09-301-2/+2
| | |
* | | Consolidate the usage of "either" and "one of" in error messagesMáté Kocsis2020-09-201-1/+1
| | | | | | | | | | | | Closes GH-6173
* | | Run tidyNikita Popov2020-09-188-26/+26
| | | | | | | | | | | | | | | This should fix most of the remaining issues with tabs and spaces being mixed in tests.
* | | Allow http tests to run in parallel by using ephemeral portsSara Golemon2020-09-171-9/+5
| | |
* | | Merge branch 'PHP-7.4'Nikita Popov2020-09-171-6/+25
|\ \ \ | |/ / | | | | | | | | | * PHP-7.4: ext/soap: Compare Set-Cookie header case-insensitively
| * | Merge branch 'PHP-7.3' into PHP-7.4Nikita Popov2020-09-171-6/+25
| |\ \ | | |/ | | | | | | | | | * PHP-7.3: ext/soap: Compare Set-Cookie header case-insensitively
| | * ext/soap: Compare Set-Cookie header case-insensitivelyGabríel Arthúr Pétursson2020-09-171-6/+25
| | | | | | | | | | | | Closes GH-6143.
* | | Consolidate new union type ZPP macro namesMáté Kocsis2020-09-111-1/+1
| | | | | | | | | | | | | | | | | | | | | They will now follow the canonical order of types. Older macros are left intact due to maintaining BC. Closes GH-6112
* | | Simplify error type filtertwosee2020-09-101-12/+2
| | | | | | | | | | | | Closes GH-6049.
* | | Improve error messages mentioning parameters instead of argumentsMáté Kocsis2020-09-091-1/+1
| | | | | | | | | | | | Closes GH-5999
* | | Add "const". Move constant strings to read-only memory.Dmitry Stogov2020-09-071-1/+1
| | |
* | | Use the canonical order of types in array|string ZPP error messagesMáté Kocsis2020-09-041-1/+1
| | |
* | | Promote warnings to exceptions in ext/soap and ext/xmlwriterMáté Kocsis2020-09-015-72/+94
| | | | | | | | | | | | Closes GH-5998
* | | Merge branch 'PHP-7.4'Matteo Beccati2020-08-182-2/+95
|\ \ \ | |/ / | | | | | | | | | * PHP-7.4: Fix #47021: SoapClient stumbles over WSDL delivered with "Transfer-Encoding: chunked"
| * | Merge branch 'PHP-7.3' into PHP-7.4Matteo Beccati2020-08-182-2/+95
| |\ \ | | |/ | | | | | | | | | * PHP-7.3: Fix #47021: SoapClient stumbles over WSDL delivered with "Transfer-Encoding: chunked"
| | * Fix #47021: SoapClient stumbles over WSDL delivered with "Transfer-Encoding: ↵Matteo Beccati2020-08-182-2/+95
| | | | | | | | | | | | chunked"
* | | Merge branch 'PHP-7.4' into masterChristoph M. Becker2020-08-181-1/+1
|\ \ \ | |/ / | | | | | | | | | * PHP-7.4: Fix HTTP response status code