Skip to content

Conversation

@kocsismate
Copy link
Member

No description provided.

@bukka
Copy link
Member

bukka commented Jan 3, 2026

The OpenSSL part LGTM

Copy link
Member

@Girgias Girgias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use <simpara> instead of <para> where able, and create an XML entity for the locale parameter and changelog of the Grapheme functions.

<term><parameter>uriParserClass</parameter></term>
<listitem>
<simpara>
The classname to use for parsing the redirection URI when a "Location" header is received in the response,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe?

Suggested change
The classname to use for parsing the redirection URI when a "Location" header is received in the response,
The classname to use for parsing the redirection URI when a <literal>"Location"</literal> header is received in the response,

<tbody>
<row>
<entry>8.5.0</entry>
<entry><methodname>SoapFault::__construct</methodname> now also expects <parameter>lang</parameter>.</entry>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<entry><methodname>SoapFault::__construct</methodname> now also expects <parameter>lang</parameter>.</entry>
<entry>
The optional parameter <parameter>lang</parameter> has been added.
</entry>

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps this changelog entry (and the other one) should describe this was necessary to be compliant to the SOAP 1.2 spec. This is what made me add this, otherwise communicating between e.g. PHP & .NET was difficult.

<tbody>
<row>
<entry>8.5.0</entry>
<entry><methodname>SoapServer::fault</methodname> now also expects <parameter>lang</parameter>.</entry>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<entry><methodname>SoapServer::fault</methodname> now also expects <parameter>lang</parameter>.</entry>
<entry>
The optional parameter <parameter>lang</parameter> has been added.
</entry>

<varlistentry>
<term><parameter>lang</parameter></term>
<listitem>
<simpara/>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ndossche could you provide a description of the new parameter?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SoapFaults in SOAP version 1.2 have a language attribute describing the (human) language that the SoapFault is written in.

So I'd describe it as follows:

The human language that the SoapFault is written in. This is only used for SOAP version 1.2.

<varlistentry>
<term><parameter>lang</parameter></term>
<listitem>
<simpara/>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ndossche ditto

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ditto

<varlistentry>
<term><parameter>digest_algo</parameter></term>
<listitem>
<simpara>The digest algorithm for OEAP padding, or &null; to use the default algorithm.</simpara>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit

Suggested change
<simpara>The digest algorithm for OEAP padding, or &null; to use the default algorithm.</simpara>
<simpara>
The digest algorithm for OEAP padding, or &null; to use the default algorithm.
</simpara>

<varlistentry>
<term><parameter>digest_algo</parameter></term>
<listitem>
<simpara>The digest algorithm for OEAP padding, or &null; to use the default algorithm.</simpara>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<simpara>The digest algorithm for OEAP padding, or &null; to use the default algorithm.</simpara>
<simpara>
The digest algorithm for OEAP padding, or &null; to use the default algorithm.
</simpara>

<row>
<entry>8.5.0</entry>
<entry>
<function>openssl_private_decrypt</function> now also accepts <parameter>digest_algo</parameter>.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<function>openssl_private_decrypt</function> now also accepts <parameter>digest_algo</parameter>.
The optional parameter <parameter>digest_algo</parameter> has been added.

<row>
<entry>8.5.0</entry>
<entry>
<function>openssl_public_encrypt</function> now also accepts <parameter>digest_algo</parameter>.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<function>openssl_public_encrypt</function> now also accepts <parameter>digest_algo</parameter>.
The optional parameter <parameter>digest_algo</parameter> has been added.

Comment on lines +89 to +94
<row>
<entry>8.5.0</entry>
<entry>
<function>grapheme_substr</function> now also accepts <parameter>locale</parameter>.
</entry>
</row>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And make this whole row tag an XML entity to reuse across all the grapheme functions with the new param.

Suggested change
<row>
<entry>8.5.0</entry>
<entry>
<function>grapheme_substr</function> now also accepts <parameter>locale</parameter>.
</entry>
</row>
<row>
<entry>8.5.0</entry>
<entry>
The optional parameter <parameter>locale</parameter> has been added.
</entry>
</row>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants