Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Squid - fix auth when LDAP/RADIUS server port is not explicitly set i… #232

Merged
merged 3 commits into from
Dec 19, 2016
Merged

Squid - fix auth when LDAP/RADIUS server port is not explicitly set i… #232

merged 3 commits into from
Dec 19, 2016

Commits on Dec 16, 2016

  1. Squid - fix auth when LDAP/RADIUS server port is not explicitly set i… 

    …n the GUI

As reported by a user @ https://forum.pfsense.org/index.php?topic=113667.msg632008#msg632008 when the port is not explicitly set in the GUI but the tags still exist in config.xml, this ads a bogus trailing semicolon which breaks Squid auth.

RADIUS would be the same case, just with trailing -p without any port. Also broken.
    doktornotor authored Dec 16, 2016
    Configuration menu
    Copy the full SHA
    d906275 View commit details
    Browse the repository at this point in the history

  2. Bump port version

    doktornotor authored Dec 16, 2016
    Configuration menu
    Copy the full SHA
    4fb5906 View commit details
    Browse the repository at this point in the history

  3. Fix typos

    doktornotor authored Dec 16, 2016
    Configuration menu
    Copy the full SHA
    40c93b2 View commit details
    Browse the repository at this point in the history