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/03 01:56] – Rename ini setting tandrerfc:throwable_string_param_max_len [2020/07/25 14:00] (current) – tandre
Line 1: Line 1:
 ====== PHP RFC: zend.exception_string_param_max_len: Configurable string length in getTraceAsString() ====== ====== PHP RFC: zend.exception_string_param_max_len: Configurable string length in getTraceAsString() ======
-  * Version: 0.4+  * Version: 0.5
   * Date: 2020-06-27   * Date: 2020-06-27
   * Author: Tyson Andre, tandre@php.net   * Author: Tyson Andre, tandre@php.net
-  * Status: Under Discussion+  * 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)
  
-==== Poll ====+Voting opened 2020-07-11 and closes 2020-07-25. A 2/3 majority is required.
  
-  * Informal poll: Interest in raising the default setting value+<doodle title="Add a new ini setting zend.exception_string_param_max_len" auth="tandre" voteType="single" closed="true"> 
 +   * Yes 
 +   * No 
 +</doodle> 
 + 
 +==== 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="true">
 +   * Yes
 +   * No
 +</doodle>
  
 ===== Changelog ===== ===== Changelog =====
Line 83: 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.txt · Last modified: 2020/07/25 14:00 by tandre

Page Tools