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

When performing a CoA, we should open a locationlog entry with the new role #2222

Closed
whitx opened this issue Apr 11, 2017 · 2 comments · Fixed by #2268
Closed

When performing a CoA, we should open a locationlog entry with the new role #2222

whitx opened this issue Apr 11, 2017 · 2 comments · Fixed by #2268
Assignees

Comments

@whitx
Copy link

whitx commented Apr 11, 2017

While doing WebAuth and a CoA, after the registration, if there is no VLAN number for the new role, the reevaluate_access is not trigger. This issue is only for CoA and not for radiusDisconnect.

@whitx whitx added this to the +1 (patch release) milestone Apr 11, 2017
@dwlfrth dwlfrth self-assigned this Apr 14, 2017
@whitx
Copy link
Author

whitx commented Apr 14, 2017

Apr 10 14:25:31 httpd.portal(31786) INFO: [mac:xx:06:f4] Connection type is WIRELESS_MAC_AUTH. Getting role from node_info (pf::role::getRegisteredRole)
Apr 10 14:25:31 httpd.portal(31786) INFO: [mac:xx:06:f4] Username was defined "xx:06:f4" - returning role 'guest' (pf::role::getRegisteredRole)
Apr 10 14:25:31 httpd.portal(31786) INFO: [mac:xx:06:f4] PID: "guest", Status: reg Returned VLAN: (undefined), Role: guest (pf::role::fetchRoleForNode)
Apr 10 14:25:31 httpd.portal(31786) WARN: [mac:xx:06:f4] No parameter guestVlan found in conf/switches.conf for the switch 10.0.0.1 (pf::Switch::getVlanByName)
Apr 10 14:25:31 httpd.portal(31786) DEBUG: [mac:xx:06:f4] No reassignment required. (pf::enforcement::_should_we_reassign_vlan)
Apr 10 14:25:31 httpd.portal(31786) DEBUG: [mac:xx:06:f4] Rendering release.html (captiveportal::PacketFence::DynamicRouting::Application::render)

@dwlfrth
Copy link
Contributor

dwlfrth commented Apr 18, 2017

Verify locationlog opening on CoA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants