Navigation Menu

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

Jump to bottom

Clients generate warnings about old style definitions but ipaddr doesn't allow ranges #581

Closed
arr2036 opened this issue Apr 9, 2014 · 2 comments
Closed

Clients generate warnings about old style definitions but ipaddr doesn't allow ranges #581

arr2036 opened this issue Apr 9, 2014 · 2 comments
Labels
defect category: a defect or misbehaviour v3.x.x meta: relates to the v3.0.x branch

Comments

@arr2036
Copy link
Member

arr2036 commented Apr 9, 2014

ipaddr should accept ranges, currently it appears to attempt to resolve ranges as hostnames.
@alandekok
Copy link
Member

sure... it currently supports "ipaddr" and "netmask", because it didn't support ranges.

tho "ipaddr = network/mask" would be useful, too. It's a low priority.

@arr2036
Copy link
Member Author

arr2036 commented Apr 9, 2014

Yes, sometimes I just open issues as reminders to do stuff, don't expect you to fix everything.

@alandekok alandekok mentioned this issue May 15, 2014
Closed
arr2036 added a commit that referenced this issue May 30, 2014
@arr2036
Switch to using fr_ipaddr_t for client prefixes, and allow ipaddr and…
9958397 
… ipaddr6 config items to be prefixes. Addresses #581.
@arr2036 arr2036 closed this as completed May 30, 2014
arr2036 added a commit that referenced this issue May 30, 2014
@arr2036
Switch to using fr_ipaddr_t for client prefixes, and allow ipaddr and…
a11748d 
… ipaddr6 config items to be prefixes. Addresses #581.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
defect category: a defect or misbehaviour v3.x.x meta: relates to the v3.0.x branch
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@alandekok @arr2036