TERASOLUNA Framework

BLogicで、DownloadInputStreamに与えたInputStreamをcloseする必要はありません。

InputStreamをDownloadInputStreamに与えてから
BLogicResultに入れてreturnした場合、
DownloadBLogicActionまで処理が返ったあとで、
FileDownloadUtil#downloadにDownloadInputStreamが渡り、
レスポンスを返し終わってから、
FileDownloadUtilがDownloadInputStream内のInputStreamをcloseします。

よって、BLogicで、DownloadInputStreamに与えたInputStreamをcloseする必要はありません。
逆に、closeしてしまうと、FileDownloadUtilが内容を読めずに、ファイルの内容をレスポンスに載せることができないかと思います。

なお、現在のBLogic実装には、気づきにくい問題が潜んでいますので、
付け加えておきます。

まず、JDBCの仕様として、BLobの有効範囲はトランザクション内に限られます。

DownloadInputStreamに与えたInputStreamは、
BLogicの処理が終わってから読み取られるため、
BLogicにAOPでトランザクションをかける一般的なトランザクション境界では、
BLobの有効範囲外(トランザクション外)でBLobの内容を読もうとしてしまいます。
(Oracleでは、どういうわけか、読み取れてしまったりしますが、コネクションをコネクションプールに返してしまった後にDBと通信しているため、
場合によっては別のトランザクションが割り込んだり、ドライバのコネクションがクローズされることもあり得ます。)

対処としては、２つ方法があります。

＜対処方法1＞
DownloadBLogicActionにAOPでトランザクションをかけ、
DownloadBLogicAction実行時にトランザクションを開始し、
ダウンロード用のBLogicでは、そのトランザクション上で動作するようにする。
(ダウンロード用のBLogicは、トランザクションAOPの対象から外すか、REQUIREDにしておく。)

こちらの方法は、設定で解決するので、楽ですが、1つだけ懸念事項があります。
DownloadBLogicAction(インタフェースなし)にAOPをかけることになり、
AOPを実現するプロキシに、CGLibプロキシが利用されることになります。
CGLibはJava8では動かないという情報があります。

＜対処方法2＞
BLogicでBLobから取得したInputStreamを読み取り、一時ファイルに書き込み、
DownloadInputStreamには、一時ファイルのFileInputStreamを渡す。

こちらの方法は、一時ファイルの削除タイミングを別途検討する必要があります。