一番不明瞭なのは、
String uploadDir=getServletContext().getRealPath("/WEB-INF/upload");
で、得られたパスのディレクトリにファイルを作れるのでしょうか。
...そこに作成するというのがそもそも間違いなのでしょうか。
シンプルな回答としては間違いです。
作れる場合もありますが作れない場合もあります。作れた場合でもその場所が恒久的に存在するとは限りません。
ServletContext#getRealPath
については Servlet仕様書(Servlet 4.0 spec(pdf) - 3.6 Path Translation Methods)やAPI docで説明されています。
In situations where the servlet container cannot determine a valid file path for these methods, such as when the Web application is executed from an archive, on a remote file system not accessible locally, or in a database, these methods must return null.
具体例を挙げると、TomcatにはunpackWARs
という設定項目がありこれをfalse
に設定すると.warファイルは展開されずにデプロイされます。
この場合の ServletContext#getRealPath
戻り値は上記引用文の通り null
です。
また別の例として、WildFlyでは.warファイルの自動展開先は一時ディレクトリであり、ServletContext#getRealPath
の結果は同じアプリケーションでもデプロイのたびに変わる可能性があります。
アップロードしたファイルが、アプリケーションを再起動すると見つからなくなってしまう、ということです。
上の話を脇に置いておいたとしても、サーバリソースが配置されているWEB-INF
ディレクトリにユーザ生成コンテンツも一緒くたに保存するというのは、運用が煩雑になるというデメリット面が大きいかと思います。