Skip to content

SNMP: add changelog for PHP 8.5 ValueError validation - #5533

Merged
jordikroon merged 7 commits into
php:masterfrom
lacatoire:changelog/snmp-8.5-bc-break
May 4, 2026
Merged

jordikroon merged 7 commits into
php:masterfrom
lacatoire:changelog/snmp-8.5-bc-break

Conversation

@lacatoire

Copy link
Copy Markdown
Member

The SNMP fetch/set functions started throwing `ValueError` in PHP 8.5 on invalid
hostname / port / timeout / retries (already documented in
`appendices/migration85/incompatible.xml`), but the per-function changelog
entries were missing.

Adds the `8.5.0` row on:

  • snmpget
  • snmpset
  • snmp2_get
  • snmp2_set
  • snmp3_get
  • snmp3_set

Same spirit as #5528.

snmpget, snmpset, snmp2_get, snmp2_set, snmp3_get and snmp3_set now throw a
ValueError when the hostname is too large, contains any null byte, when the
port is negative or greater than 65535, or when the timeout or retries values
are lower than -1 or too large. This is documented in
appendices/migration85/incompatible.xml but the per-function changelog entries
were missing.

@jordikroon jordikroon left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Mostly fine and accurate. Just a minor suggestion on the hostname length.

Comment thread reference/snmp/functions/snmp2-get.xml Resolved
Comment thread reference/snmp/functions/snmp2-set.xml Resolved
Comment thread reference/snmp/functions/snmp3-get.xml Resolved
Comment thread reference/snmp/functions/snmp3-set.xml Resolved
Comment thread reference/snmp/functions/snmpget.xml Resolved
Comment thread reference/snmp/functions/snmpset.xml Resolved
lacatoire and others added 6 commits May 4, 2026 21:41
Co-authored-by: Jordi Kroon <jordi@jordikroon.nl>
Co-authored-by: Jordi Kroon <jordi@jordikroon.nl>
Co-authored-by: Jordi Kroon <jordi@jordikroon.nl>
Co-authored-by: Jordi Kroon <jordi@jordikroon.nl>
Co-authored-by: Jordi Kroon <jordi@jordikroon.nl>
Co-authored-by: Jordi Kroon <jordi@jordikroon.nl>
@jordikroon
jordikroon merged commit dd22c78 into php:master May 4, 2026
2 checks passed
@lacatoire
lacatoire deleted the changelog/snmp-8.5-bc-break branch May 4, 2026 20:45
lacatoire added a commit to php/doc-fr that referenced this pull request May 5, 2026
* SNMP: ajout du changelog PHP 8.5 pour la validation ValueError

Synchronise snmpget, snmpset, snmp2_get, snmp2_set, snmp3_get et snmp3_set
avec doc-en@dd22c782 (PR php/doc-en#5533): ajout de l'entrée changelog 8.5.0
mentionnant la ValueError lancée pour un hostname trop long, un port hors
plage, ou un timeout/retries hors bornes.

Closes #2782

* SNMP: passe maintainer à lacatoire
jordikroon added a commit to Narkunan/doc-en that referenced this pull request May 17, 2026
* SNMP: add changelog for PHP 8.5 ValueError validation

snmpget, snmpset, snmp2_get, snmp2_set, snmp3_get and snmp3_set now throw a
ValueError when the hostname is too large, contains any null byte, when the
port is negative or greater than 65535, or when the timeout or retries values
are lower than -1 or too large. This is documented in
appendices/migration85/incompatible.xml but the per-function changelog entries
were missing.

Co-authored-by: Jordi Kroon <jordikroon@me.com>
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.

2 participants