Comment 13 for bug 1811817

Revision history for this message
Mauricio Faria de Oliveira (mfo) wrote :

I can confirm the current fix works, but had a concern I'd like to bring to another reviewer for a second option, at least.
(Added to 'Other Info' for now).

 * This is actually (re)introducing SSL support back into the package,
   which was not available at Focal release, so this could be seen as
   a feature.

   However, considering the lack of SSL breaks the *non-SSL* FTP case
   (no guard in ftp_connect()) this could also be considered a bug-fix,
   as the package only has FTP (non-SSL) and SFTP (SSL) functionality.

   Perhaps fixing the SSL guards properly to prevent the segfault in
   non-SSL FTP mode is the minimal/actual bug-fix, but then there is
   little left for this package (and its docs/man would need updates).