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

Fix GenericVPN switch module #8270

Merged
merged 1 commit into from
Sep 11, 2024
Merged

Fix GenericVPN switch module #8270

merged 1 commit into from
Sep 11, 2024

Conversation

fdurand
Copy link
Member

@fdurand fdurand commented Aug 30, 2024

Description

Don't create a mac address if the radius attribute is empty.

Impacts

GenericVPN switch module

Issue

  • Don't generate all the time a mac address when using the GenericVPN switch module (The logic is going in the wrong path)

Delete branch after merge

YES

NEWS file entries

Bug Fixes

  • Don't generate all the time a mac address when using the GenericVPN switch module

@fdurand fdurand added this to the +1 (patch release) milestone Aug 30, 2024
@satkunas satkunas merged commit 7324cb0 into devel Sep 11, 2024
13 checks passed
satkunas pushed a commit that referenced this pull request Sep 12, 2024
@fdurand
Fix GenericVPN switch module (#8270)
5a563d9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@satkunas satkunas satkunas approved these changes

@jrouzierinverse jrouzierinverse Awaiting requested review from jrouzierinverse

Assignees

@jrouzierinverse jrouzierinverse

Labels
Priority: High Status: For review Type: Bug
Projects
None yet
Milestone
+1 (patch release)
Development

Successfully merging this pull request may close these issues.
3 participants
@fdurand @satkunas @jrouzierinverse