Project

General

Profile

Actions

Bug #3561

closed

PPTP VPN broken without RADIUS - always requires RADIUS server configuration.

Added by Jānis Veinbergs about 10 years ago. Updated over 9 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
PPTP
Target version:
-
Start date:
04/01/2014
Due date:
% Done:

0%

Estimated time:
Plus Target Version:
Release Notes:
Affected Version:
2.1
Affected Architecture:

Description

Upgrading from 1.2.3 to 2.1 and importing config, that includes PPTP VPN, I had problems with bringing it up, because It required me to specify RADIUS server which I don't use. If I unchecked "Use RADIUS server for authentication", I would get error: "A valid RADIUS server address must be specified." Had to fix line 102 in vpn_pptp.php to override this behaviour.

Sometimes the PHP script errored on line 169 with "Cannot unset string offsets", but sorry - I forgot how to replicate it, because was hurrying to finish the upgrade and was just trying to uncheck/check various checkboxes on PPTP server. Line 169:

unset($pptpcfg['radius']['server']['enable']);

It looks like long ago I was experimenting with RADIUS server, but left it turned off, as in the config file, I had a RADIUS server address specified:

<pptpd>
    <redir/>
    <localip>someIP1</localip>
    <remoteip>someIP2</remoteip>
    <radius>
        <server>someIP3</server>
        <secret>scret</secret>
        <accounting/>
    </radius>
    <wins>someIP4</wins>
    <req128/>
    <user>
        <name><!--User--></name>
        <ip/>
        <password><!--Password--></password>
    </user>
    <!-- More users -->

    <mode>server</mode>
</pptpd>

Too bad that I found it out the hard way and was thinking that PPTP in 2.1 is totally broken.

Actions #1

Updated by Chris Buechler over 9 years ago

  • Status changed from New to Resolved

the original issue here was the result of a config upgrade issue that's since been fixed at some point. I tested restoring 1.2.3 configs to 2.2, and every possible PPTP scenario is converted correctly.

Actions

Also available in: Atom PDF