#538 - LOG_DIR not set causing FileNotFoundException

[MarkC0x]

Noticed that this same LOG_DIR not set causing FileNotFoundException behavior was still happening in the latest 5.5.x version of Rest Proxy.

Created a simple PR to correctly set the base log directory for the logs.

[ghost]

@confluentinc It looks like @MarkC0x just signed our Contributor License Agreement. 👍

Always at your service,

clabot

[rigelbm]

[[ -z "${LOG_DIR+x}" ]] && LOG_DIR="$base_dir/logs"

[rigelbm]

BTW, not sure if $base_dir/logs is a better default log folder than, for example, /var/log/kafka-rest. AK uses /var/log/kafka by default, I think.

[MarkC0x]

Thanks for reviewing @rigelbm -- I used the same $base_dir/logs default folder code present in both AK and CP kafka-run-class files, so that things remained consistent. I can change the logic to your modified version though if you prefer?

[cla-assistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

---

Mark Cox seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}