rfc:throwable_string_param_max_len

Differences

This shows you the differences between two versions of the page.

Link to this comparison view

Both sides previous revisionPrevious revision
Next revision		Previous revision
rfc:throwable_string_param_max_len [2020/07/11 18:17] – Start vote tandrerfc:throwable_string_param_max_len [2020/07/25 14:00] (current) – tandre
Line 3: Line 3:
   * Date: 2020-06-27   * Date: 2020-06-27
   * Author: Tyson Andre, tandre@php.net   * Author: Tyson Andre, tandre@php.net
-  * Status: Voting +  * Status: Implemented 
   * First Published at: https://wiki.php.net/rfc/throwable_string_param_max_len   * First Published at: https://wiki.php.net/rfc/throwable_string_param_max_len
   * Implementation: https://github.com/php/php-src/pull/5769   * Implementation: https://github.com/php/php-src/pull/5769
Line 66: Line 66:
 Application may be unexpectedly relying on the hardcoded limit of 15 to avoid logging sensitive information such as full urls, full paths, or full file contents. Application may be unexpectedly relying on the hardcoded limit of 15 to avoid logging sensitive information such as full urls, full paths, or full file contents.
  
-===== Proposed Voting Choices =====+===== Vote =====
  
 Add a new ini setting ''zend.exception_string_param_max_len'' as described in the RFC. (Yes/No, requiring 2/3 majority) Add a new ini setting ''zend.exception_string_param_max_len'' as described in the RFC. (Yes/No, requiring 2/3 majority)
Line 72: Line 72:
 Voting opened 2020-07-11 and closes 2020-07-25. A 2/3 majority is required. Voting opened 2020-07-11 and closes 2020-07-25. A 2/3 majority is required.
  
-<doodle title="Add a new ini setting zend.exception_string_param_max_len" auth="tandre" voteType="single" closed="false">+<doodle title="Add a new ini setting zend.exception_string_param_max_len" auth="tandre" voteType="single" closed="true">
    * Yes    * Yes
    * No    * No
Line 79: Line 79:
 ==== Poll ==== ==== Poll ====
  
-<doodle title="Informal poll: Interest in raising the default string parameter max length from 15 bytes in future RFCs" auth="tandre" voteType="single" closed="false">+<doodle title="Informal poll: Interest in raising the default string parameter max length from 15 bytes in future RFCs" auth="tandre" voteType="single" closed="true">
    * Yes    * Yes
    * No    * No
Line 92: Line 92:
 0.4: Update external links, formatting. 0.4: Update external links, formatting.
  
-0.5: Rename from ''throwable_string_param_max_len'' to ''zend.exception_string_param_max_len''+0.5: Rename from ''throwable_string_param_max_len'' to ''zend.exception_string_param_max_len''. Add reference to RFC thread. Rename "Proposed Voting Choices" section to "Vote"
  
 ===== References ===== ===== References =====
  
-https://externals.io/message/110717 "Making the hardcoded string length limit of Throwable->getTraceAsString() configurable"+https://externals.io/message/110717 "Making the hardcoded string length limit of %%Throwable->getTraceAsString()%% configurable"
  
 +https://externals.io/message/110744 "[RFC] throwable_string_param_max_len: Configurable string length in getTraceAsString()"
  
 ===== Appendix ===== ===== Appendix =====
rfc/throwable_string_param_max_len.1594491455.txt.gz · Last modified: 2020/07/11 18:17 by tandre

Page Tools