not related directly to this patch, but should the resource attr be ftp_passive_mode instead of "active_mode"?   i'm more used to thinking about ftp as being in passive mode or not...  thinking about "active mode" makes me have to translate that setting this to false is the same as setting passive mode to true...