Bug #23433
closed
- Related to Bug #23031: ISC DHCP new parser is still unable to parse DUID added
The literal regex is still broken.
def server_duid
Rsec::Fail.reset
keyword = word('server-duid').fail 'keyword_server_duid'
anything = /[^;,{}\s]+/.r
literal = /"([^"]|\")*"/.r
fails on
server-duid "\000\001\000\001\"vt\346\000PV\225S\347";
Changing the regex to
literal = /"([^"]|\\\")*"/.r
Though you could just literally do /".*"/ and rely on greedy regex match to fix the issue and simplify the code.
- Status changed from New to Ready For Testing
- Assignee set to Anonymous
- Pull request https://github.com/theforeman/smart-proxy/pull/583 added
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
- Translation missing: en.field_release set to 330
I guess this should be pulled into 1.18 release
1.17 branch would be even better. Unfortunately, I can't update the release field.
- Pull request https://github.com/theforeman/smart-proxy/pull/588 added
- Category set to DHCP
- Translation missing: en.field_release changed from 330 to 360
Requesting backport into 1.17, this is an important one: #23784
I concur, this is an important patch. For anyone wanting to try the patch on 1.17:
- Related to Bug #24978: ISC DHCP parser does not support DDNS entries added
Also available in: Atom
PDF
Updated by 
Updated by 
Updated by 
Updated by 
Updated by 
Updated by 
Updated by