Bug 38753 - univention-ftp: broken firewall
univention-ftp: broken firewall
Status: RESOLVED WONTFIX
Product: UCS
Classification: Unclassified
Component: General
UCS 4.0
Other Linux
: P5 normal (vote)
: ---
Assigned To: UCS maintainers
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2015-06-23 13:04 CEST by Philipp Hahn
Modified: 2019-01-03 07:17 CET (History)
2 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 3: Simply Wrong: The implementation doesn't match the docu
Who will be affected by this bug?: 1: Will affect a very few installed domains
How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.034
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Philipp Hahn univentionstaff 2015-06-23 13:04:03 CEST
1. univention-ftp provides no proper LDAP integration
2. The firewall configuration is broken:
   - In active mode the server opens an *outgoing* connection from TCP port 20
   - In passive mode the server requires to open unprivileged ports >=1024 for incoming connections.

In any case opening port 20 for *incoming* connections is wrong.
Using "nf_conntrack_ftp.ko" should be considered as well.

<https://de.wikipedia.org/wiki/File_Transfer_Protocol#Verbindungsarten>

Perhaps the package should be removed?
Comment 1 Stephan Hendl 2015-07-31 07:53:17 CEST
Well, the wiki document "LDAP authentication with ProFTP" is not valid for ucs-4.x anymore. The proftpd team changed the ldap configuration completely. I'd to change the config lines from the wiki to the following:

# Only use LDAP Auth
AuthOrder                       mod_ldap.c
<IfModule mod_ldap.c>
# The LDAP server istself (with or without TLS)
LDAPServer <fqdn of the DC master/backup/slave>:7389
LDAPBindDN <DN of the authentication account> <Password of the authentication account>

# GID to name in dir listing
LDAPGroups cn=groups,$ldap_base

# UID to name in dir listing
LDAPUsers cn=users,$ldap_base (uid=%u) (uidNumber=%u)

# TLS on/off
LDAPUseTLS on

# Create homedir if not exists
CreateHome on 711 skel /etc/skel
LDAPGenerateHomedir on
</IfModule> 

That works...

@Philipp: don't remove, please!
Comment 2 Philipp Hahn univentionstaff 2018-02-08 13:27:28 CET
base/univention-pam/conffiles/etc/pam.d/ftp is not used by ProFTPd - it uses /etc/pam.d/proftpd which is not UCRified.

Again: time for removal?
Comment 3 Stefan Gohmann univentionstaff 2019-01-03 07:17:26 CET
This issue has been filled against UCS 4.0. The maintenance with bug and security fixes for UCS 4.0 has ended on 31st of May 2016.

Customers still on UCS 4.0 are encouraged to update to UCS 4.3. Please contact
your partner or Univention for any questions.

If this issue still occurs in newer UCS versions, please use "Clone this bug" or simply reopen the issue. In this case please provide detailed information on how this issue is affecting you.