Close FileOutputStream in downloadFileWithProgress

[tejasae-afk]

downloadFileWithProgressBar is close, but the resource opened there can leak if downloadFileWithProgressBar exits on an error path. I moved the allocation into try-with-resources so cleanup happens on every exit path.

[CLAassistant]

All committers have signed the CLA.

[hohwille]

@tejasae-afk thank you for contributing to IDEasy and providing this PR.
However, I slightly disagree with your analysis.
According to how try-with-resource works:

• https://docs.oracle.com/javase/tutorial/essential/exceptions/tryResourceClose.html
• https://www.baeldung.com/java-try-with-resources

all closable resources opened by the try block will be guaranteed to be closed if they could be created.
This includes all error paths even those creating further resources in the try block.
Nesting try-with-resources blocks is IMHO not needed here.

BTW: It seems your code change does not even compile and was thus untested.

If you are still thinking I am wrong here, please explain further and fix your code.
Then I am more than willing to merge.