struts-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yasser Zamani (JIRA)" <>
Subject [jira] [Commented] (WW-5009) EmptyStackException in JSON plugin due to concurrency
Date Sat, 02 Feb 2019 10:16:00 GMT


Yasser Zamani commented on WW-5009:

Thank you [~JCjA492kH6347Vb10Ja53]! But I think it's not a nice idea and I think it doesn't
have a logical evidence to force user a synchronization proposed above (when it's internally
resolvable). Despite how much refactor/modifications are needed, I think instead we should
fix {{DefaultJSONWriter}}. Did you mean you already have seen that it isn't resolvable and
using java {{synchronized}} feature is the only available solution? I feel it should be already
fixable to be thread-safe.

> EmptyStackException in JSON plugin due to concurrency
> -----------------------------------------------------
>                 Key: WW-5009
>                 URL:
>             Project: Struts 2
>          Issue Type: Bug
>          Components: Plugin - JSON
>    Affects Versions: 2.5.20
>         Environment: JDK 1.8_0_191
> Tomcat 9.0.12
>            Reporter: Eduardo Quintanilla
>            Assignee: Yasser Zamani
>            Priority: Critical
>              Labels: json, json-rpc
>             Fix For: 2.5.21, 2.6
>         Attachments: logs.txt,
> EmptyStackException in JSON plugin due to concurrency
> Note: The issue was found in an app that was upgraded from version 2.3.34 
> Logs: [^logs.txt]
> I created a sample application that should help reproducing the error.
> [^]
> The error happened opening the app (localhost/struts-json-example) in 4 different browsers
/ sessions.
> As a workaround I have overridden the JSONInterceptor so a new instance of JSONUtil is
used with every request and the issue is not happening anymore. I suppose there is a performance
cost with that change but for now it works.

This message was sent by Atlassian JIRA

View raw message