*** gregraka is now known as gregraka-afk | 13:04 | |
*** gregraka-afk is now known as gregraka | 14:37 | |
opendevreview | Don Kehn proposed openstack/designate master: Clarifies the zone import error message https://review.opendev.org/c/openstack/designate/+/835651 | 15:34 |
---|---|---|
frickler | does anyone use CAA with a flag != 0? according to the RFC and some docs, the allowed value != 0 should be 128, but we only allow 0 or 1 | 16:03 |
johnsom | https://www.irccloud.com/pastebin/I88lC4x5/ | 16:13 |
johnsom | The field is a byte, so we should support more than just 1, but the spec says 0/1. Annoyingly, below, the example uses 128. | 16:14 |
johnsom | Actually, I wonder if this is just the msb issue. | 16:15 |
johnsom | Yeah, someone probably didn't read all of the MSB info. So, bit 0 set to 1 is 128 in integer | 16:17 |
johnsom | The way DNS defines the MSB | 16:18 |
johnsom | https://github.com/openstack/designate/blob/master/designate/objects/rrdata_caa.py#L29 | 16:21 |
johnsom | 100% a bug | 16:21 |
johnsom | I opened a bug to track it: https://bugs.launchpad.net/designate/+bug/1966982 | 16:34 |
frickler | thx johnsom | 17:40 |
dkehn | johnsom https://review.opendev.org/c/openstack/designate/+/829388 need your attn | 21:20 |
dkehn | eandersson, https://review.opendev.org/c/openstack/designate/+/829388 need another pass | 21:21 |
johnsom | ack | 21:21 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!