gregh3269 commented on pull request #435:
URL: https://github.com/apache/struts/pull/435#issuecomment-689366835


   I did consider a struts.messages.error.file.empty, but the patch was more 
related to the incorrect message rather than the semantics of what 
JakartaMultiPartRequest does, there maybe other reasons for the null 
storeLocation  if/IOException.
   From the snip the logging is odd as the storeLocation is null for an empty 
file (I have enctype="multipart/form-data" for the upload form) so never gets 
to the  IOException.  There may be other implementations?
   
   @lukaszlenart For the PR on 2.5.x I do not have any notes on how to do this 
:-(


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to