weblegacy / struts1

Struts1-Upgrade to current technology
https://weblegacy.github.io/struts1/
Apache License 2.0
31 stars 5 forks source link

IOException in commons-fileupload2 method #17

Closed lukamalovic closed 8 months ago

lukamalovic commented 10 months ago

In the commons-fileupload2 dependency, the getString(Charset cs) method in FileItem now throws an IOException, which it did not previously. I was able to mitigate this in the weblegacy-chain branch by propagating the error anywhere the method was referenced, although I imagine going forward a more robust solution would be preferred, which is why I'm making note of it here.

ste-gr commented 10 months ago

Hello @lukamalovic,

I have solved the problem with c0532cd. There were also some other changes for fileupload2. I think it is not necessary to route the IOException. But have a look at my changes.

Greetings Stefan