Skip to content

Commit

Permalink
T4726: Remove accel-ppp RADIUS vendor validators
Browse files Browse the repository at this point in the history
The vendor name could contain Uppercase or lowercase symbols and
not rely on the dictionary name but on dictionary value

  / # cat /usr/share/freeradius/dictionary.cisco | grep -i vendor
  VENDOR		Cisco				9

Another example
  VENDOR          Alcatel-IPD               6527

This way if we use `vendor=cisco` instead of `vendor=Cisco` it
will not work at all

Delete vendor validators
  • Loading branch information
sever-sever committed Nov 1, 2023
1 parent cb2f72d commit bbc7cab
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 19 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -13,12 +13,6 @@
<leafNode name="vendor">
<properties>
<help>Vendor dictionary</help>
<completionHelp>
<list>alcatel cisco microsoft mikrotik</list>
</completionHelp>
<constraint>
<validator name="accel-radius-dictionary" />
</constraint>
</properties>
</leafNode>
<leafNode name="enable">
Expand Down
13 changes: 0 additions & 13 deletions src/validators/accel-radius-dictionary

This file was deleted.

0 comments on commit bbc7cab

Please sign in to comment.