[ 
https://issues.apache.org/jira/browse/WW-5819?focusedWorklogId=1050529&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-1050529
 ]

ASF GitHub Bot logged work on WW-5819:
--------------------------------------

                Author: ASF GitHub Bot
            Created on: 27/Oct/26 06:19
            Start Date: 27/Oct/26 06:19
    Worklog Time Spent: 10m 
      Work Description: lukaszlenart merged PR #2062:
URL: https://github.com/apache/struts/pull/2062




Issue Time Tracking
-------------------

    Worklog Id:     (was: 1050529)
    Time Spent: 1h  (was: 50m)

> Log a validator param that fails type conversion instead of ignoring it 
> silently
> --------------------------------------------------------------------------------
>
>                 Key: WW-5819
>                 URL: https://issues.apache.org/jira/browse/WW-5819
>             Project: Struts 2
>          Issue Type: Improvement
>          Components: XML Validators
>            Reporter: Lukasz Lenart
>            Priority: Major
>             Fix For: 7.5.0
>
>          Time Spent: 1h
>  Remaining Estimate: 0h
>
> h2. Problem
> {{DefaultValidatorFactory}} builds each validator through 
> {{ObjectFactory.buildValidator}}, which sets the configured params with type 
> conversion errors swallowed. A param that cannot be converted is left unset, 
> and nothing is logged. For a range bound, that turns the check off without 
> any trace:
> {code:xml}<field-validator type="localdate">
>     <param name="min">01/02/2026</param>  <!-- not an ISO date: min stays 
> null -->
> </field-validator>
> {code}
> h2. Proposal
> Keep the current behaviour (the validator is still built and the param is 
> still left unset), but log the swallowed conversion error. The message names 
> the validator type, the param, the value and the cause:
> * WARN by default
> * ERROR in devMode
> h2. Scope
> A logging-only change with no behaviour change, so it is safe for 7.5.0. 
> Failing the configuration instead is WW-5818 (8.0.0).
> h2. Tests
> * an unconvertible param on an XML-configured validator produces the log 
> entry at the expected level, and the validator is still built
> * a valid param produces no entry



--
This message was sent by Atlassian Jira
(v10.3.26#10030026)

Reply via email to