#5608 closed enhancement (complete)

Must be possible to specify extra attributes (e.g. password) for Access-Request

Reported by: tomek Owned by: tomek
Priority: high Milestone: Kea1.4
Component: hook-radius Version: git
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity: N/A
Sub-Project: DHCP Feature Depending on Ticket:
Estimated Difficulty: 0 Add Hours to Ticket: 0
Total Hours: 0 Internal?: no

Description

A customer asked for implementation of a feature that would allow to send extra attributes in Access-Request. In particular, Password should be supported.

For details see support ticket #12745, issue 2.

Subtickets

Change History (7)

comment:1 Changed 19 months ago by tomek

  • Summary changed from Must be possible to specify extra attributes (r.g. password) for Access-Request to Must be possible to specify extra attributes (e.g. password) for Access-Request

comment:2 Changed 19 months ago by fdupont

Note this feature is already supported... But IMHO we should keep this ticket because it is poorly documented.

comment:3 Changed 19 months ago by tomek

  • Owner set to tomek
  • Status changed from new to assigned

comment:4 Changed 19 months ago by tomek

  • Owner changed from tomek to Unassigned
  • Status changed from assigned to reviewing

Doc written, please review. Updated docs on trac5608 in kea repo.

comment:5 Changed 19 months ago by fdupont

  • Owner changed from Unassigned to fdupont

comment:6 Changed 19 months ago by fdupont

  • Owner changed from fdupont to tomek

Fixed the example (moved host reservation blocks as I did in #5605, swapped Info and Token, etc). Now it loads.

Note the reason expression values are restricted to access is because it requires the query message to be evaluated, something that callout points used for accounting do not provide.

Updated the example server part in the guide to match the example file.

The reason for Host Cache is plainly wrong (not false but not related to the real reason which is that the hook uses select subnet callout points to call the RADIUS server and host cache to cache the retrieved information for further uses).

I suggest to add a note about current restrictions (no shared network, no host database backend).

Please pull and review my changes.

comment:7 Changed 19 months ago by fdupont

  • Resolution set to complete
  • Status changed from reviewing to closed

Cherry-picked the commits into trac5605a so closing this ticket.

Note: See TracTickets for help on using tickets.