<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jun 23, 2023, 17:47 Ondrej Zajicek <<a href="mailto:santiago@crfreenet.org">santiago@crfreenet.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, Jun 12, 2023 at 01:08:15PM +0200, Alexander Zubkov via Bird-users wrote:<br>
> Hi,<br>
> <br>
> Currently one can use only a predefined set of advertised options in radv<br>
> protocol, that are supported by bird's configuration. It would be<br>
> convenient to be able to specify other possible options at least manually<br>
> as a blob so one should not wait until it is supported in the code,<br>
> released, etc.<br>
<br>
Hi<br>
<br>
Merged:<br>
<br>
<a href="https://gitlab.nic.cz/labs/bird/-/commit/9c81250c04798fd274ae9d77380e93b941ac2d7f" rel="noreferrer noreferrer" target="_blank">https://gitlab.nic.cz/labs/bird/-/commit/9c81250c04798fd274ae9d77380e93b941ac2d7f</a><br>
<br>
Hopefully there is no requirement for RA options to be sorted by type, at<br>
least i did not find it in RFC 4861.<br>
<br>
The only objection from me is that 'other type' option name is kind of<br>
non-descriptive, does not indicate it is related to RA options (nor it is<br>
implicated by context). I do not really have a good idea for alternative,<br>
perhaps just 'custom option'? What do you think?<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Yes, I was thinking about "custom" too. But it introduces a new keyword. So I tried too choose something suitable from available keywords. But if it is not a problem, I would prefer "custom" too as more descriptive.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Could you prepare a patch for documentation?<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Sure, just will wait for the final decision about the syntax ("other" vs "custom").</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
BTW, why not to use WALK_LIST() in radv_prepare_custom()?<br>
(just noticed in now)<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">That is a good question. Actually I just used the structure of the similar function for the predefined option and didn't thought much about it. Now that you pointed that out, I remember that macro from the other parts of the code. And it seems reasonable to use it here, and probably in the "source" function too. I can prepare a patch for that.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
-- <br>
Elen sila lumenn' omentielvo<br>
<br>
Ondrej 'Santiago' Zajicek (email: <a href="mailto:santiago@crfreenet.org" target="_blank" rel="noreferrer">santiago@crfreenet.org</a>)<br>
OpenPGP encrypted e-mails preferred (KeyID 0x11DEADC3, <a href="http://wwwkeys.pgp.net" rel="noreferrer noreferrer" target="_blank">wwwkeys.pgp.net</a>)<br>
"To err is human -- to blame it on a computer is even more so."<br>
</blockquote></div></div></div>