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

radius: dictionary for privacyIDEA-Serial is missing #29

Open
cornelinux opened this issue Feb 14, 2020 · 3 comments
Open

radius: dictionary for privacyIDEA-Serial is missing #29

cornelinux opened this issue Feb 14, 2020 · 3 comments
Assignees

Comments

@cornelinux
Copy link
Member

The package privacyidea-radius is missing the file dictionary.netknights, that specifies the RADIUS attribute privacyIDEA-Serial.

# -*- text -*-
#
# 	privacyIDEA dictionary
#

VENDOR		NetKnights			44929

#  Backwards compatibility.
BEGIN-VENDOR	NetKnights

ATTRIBUTE	privacyIDEA-Serial					1	string

END-VENDOR	NetKnights
@cornelinux
Copy link
Member Author

cornelinux commented Feb 29, 2020

We could add this file to /usr/share/freeradius/dictionary.netknights. In this case it would be available but not active, yet.
The admin could then include it either systremwide in /usr/share/freeradius/dictionary or in the local /etc/freeradius/3.0/dictionary.

As another option we could immediately modify the local dictionary file /etc/freeradius/3.0/dictionary, which does not sound that elegant.

As most installations probably will not use the privacyIDEA-Serial I think simply shiipping the file would be enough.
Also, the attribute is not used in rlm_perl.ini anymore per default, so not activating the dectionary would totally fine, since no error is contained in the log file.

@superit20
Copy link

Maybe default /etc/privacyidea/rlm_perl.ini should contain
[Mapping]
serial = privacyIDEA-Serial
after provacyIDEA installation?
That will be a straightforward soultion.

@cornelinux
Copy link
Member Author

@superit20 This entry is already contained. But the problem is, that privacyIDEA-Serial is not defined as a RADIUS Attribute!

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

No branches or pull requests

3 participants