Opened 4 years ago

Closed 4 years ago

#4079 closed enhancement (worksforme)

Simplify option declaration

Reported by: shane Owned by: tomek
Priority: medium Milestone: Outstanding Tasks
Component: Unclassified 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 (last modified by tomek)

As far as I know, right now options are defined like this:

            "option-data": [
                {
                    "name": "domain-name-servers",
                    "code": 6,
                    "space": "dhcp4",
                    "csv-format": true,
                    "data": "172.29.2.1"
                },
                {
                    "name": "routers",
                    "code": 3,
                    "space": "dhcp4",
                    "csv-format": true,
                    "data": "172.29.2.1"
                }
            ]

It might be nice if there were some defaults:

  • "csv-format" could just default to true
  • "space" could default to "dhcp4" in the "Dhcp4" section or "dhcp6" in the "Dhcp6" section
  • If only "name" or "code" is used, it could default to the matching value

Subtickets

Change History (5)

comment:1 Changed 4 years ago by fdupont

I had to define an option for 4o6 (if you know the RFC you know which one) and I can confirm the current syntax can be more user friendly (but as it works today IMHO either the priority should be lower or it should be postponed to > 1.0).

comment:2 Changed 4 years ago by hschempf

  • Milestone changed from Kea-proposed to DHCP Outstanding Tasks

Per team meeting Oct 14, move to outstanding (we aren't accepting new tickets in 1.0 unless they are urgent or related to client class)

comment:3 Changed 4 years ago by tomek

  • Milestone changed from DHCP Outstanding Tasks to Outstanding Tasks

Milestone renamed

comment:4 Changed 4 years ago by tomek

  • Description modified (diff)
  • Owner set to tomek
  • Status changed from new to reviewing

comment:5 Changed 4 years ago by tomek

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

It was implemented a long time ago. Not exactly sure when, but it's present in current 1.1.0-beta. Closing ticket.

Note: See TracTickets for help on using tickets.