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.

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

Feature/cambium ap modules #3282

Merged
merged 7 commits into from
Jun 14, 2018
Merged

Feature/cambium ap modules #3282

merged 7 commits into from
Jun 14, 2018

Conversation

atran-inverse
Copy link
Contributor

Description

New Cambium AP module for 802.1x, MAC Auth and Web auth.
Added documentation

Impacts

Allows PF to use Cambium AP

Delete branch after merge

YES


Next, go to *Configure -> WLAN*, and click on *Add WLAN*. Give it the desired ID, and enter your SSID, default VLAN, and select *WPA2 Enterprise* for Security.

image::docs/images/cambium-dot1x-basic.png[scaledwidth="100%",alt="Cambium WLAN Basic"]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The image is missing.


=head1 SYNOPSIS

Implements a Cambium AP which supports 802.1x in wireless
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mac-auth and webauth

sub supportsWirelessMacAuth { return $TRUE; }
sub supportsExternalPortal { return $TRUE; }
sub supportsWebFormRegistration { return $TRUE }

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sub inlineCapabilities { return ($MAC,$PORT); }

synchronize_locationlog => $TRUE,
);
use Data::Dumper;
$logger->info(Dumper(\%params));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove


sub parseExternalPortalRequest {
my ( $self, $r, $req ) = @_;
my $logger = $self->logger;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove

ssid => $req->param('ga_ssid'),
synchronize_locationlog => $TRUE,
);
use Data::Dumper;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove


use pf::config qw(
$MAC
$SSID
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

useless

use pf::config qw(
$MAC
$SSID
$WIRELESS_MAC_AUTH
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

useless

use warnings;

use POSIX;
use Try::Tiny;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

useless

use strict;
use warnings;

use POSIX;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

useless

$WIRELESS_MAC_AUTH
);
use pf::constants;
use pf::node;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

useless

cnPilot E410
^^^^^^^^^^^^

802.1x
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

802.1x -> 802.1X everywhere


image::docs/images/cambium-dot1x-basic.png[scaledwidth="100%",alt="Cambium WLAN Basic"]

In the *Radius Server* tab, enter the management IP address of PacketFence (VIP in case of a cluster) and the Radius secret for Authentication and Accounting servers.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Radius -> RADIUS

<script src="/content/autosubmit.js" type="text/javascript"></script>
];

$logger->debug("Generated the following html form : ".$html_form);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMO no, it can be usefull

@extrafu extrafu added this to the PacketFence-8.1 milestone Jun 13, 2018
@fdurand fdurand merged commit f5827af into devel Jun 14, 2018
fdurand added a commit that referenced this pull request Jun 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants