Github user aledsage commented on the issue:
https://github.com/apache/brooklyn-server/pull/918
Agree with @tbouron that supporting `auth` for ftp would be a nice thing to
have in the future, but I don't think it's required for this PR.
By the way @sjcorbett, a pleasure to review - nice clear code, and well
tested. Merging now.---
