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

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

                Author: ASF GitHub Bot
            Created on: 01/Oct/25 12:00
            Start Date: 01/Oct/25 12:00
    Worklog Time Spent: 10m 
      Work Description: MFAshby opened a new pull request, #1377:
URL: https://github.com/apache/struts/pull/1377

   This can mask configuration errors and result in a non-working application.




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

            Worklog Id:     (was: 985318)
    Remaining Estimate: 0h
            Time Spent: 10m

> Bad interceptor configuration is masked
> ---------------------------------------
>
>                 Key: WW-5578
>                 URL: https://issues.apache.org/jira/browse/WW-5578
>             Project: Struts 2
>          Issue Type: Bug
>            Reporter: Martin
>            Priority: Major
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> Interceptors which have initialization errors; e.g. maybe they are supplied 
> by Spring and they have missing dependencies, are simply skipped instead of 
> throwing a configuration error. See this code in 
> org.apache.struts2.config.providers.InterceptorBuilder#constructInterceptorReference
> {code:java}
>                 try {
>                     Interceptor inter = 
> objectFactory.buildInterceptor(config, refParams);
>                     result.add(new InterceptorMapping(refName, inter, 
> refParams));
>                 } catch (ConfigurationException ex) {
>                           LOG.warn(new ParameterizedMessage("Unable to load 
> config class {} at {} probably due to a missing jar, which might be fine if 
> you never plan to use the {} interceptor",
>                             config.getClassName(), ex.getLocation(), 
> config.getName()), ex);
>                 }
> {code}
> This could result in an only-partially working application. Failing fast is 
> generally preferred in order to be certain that everything is actually 
> working as its been configured.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to