Hi, On Tue, May 31, 2016 at 11:54:53AM -0400, Selva Nair wrote: > On Tue, May 31, 2016 at 4:48 AM, Gert Doering <[email protected]> wrote: > > > Thinking about this a bit more, without having studied the code - what > > do you think about having a third category [..] > > "ignore" would be "if that option shows up, log the fact that we ignored > > it, and go on", while "reject" would be "if this shows up, something bad > > is happening, so complain loudly, abort the connection and try the next > > alternative". > > Interesting. At the very least it may be worth doing a v2 to make adding > new action types easier.
Do you foresee more different action types? My imagination ends at
"accept, ignore, refuse-with-error"...
> Suppose we do make "reject" to not just remove the option but reject the
> server. What to do if no alternative remotes exist --- just SIGHUP and hope
> that next try will succeed or something like management-hold will keep the
> connection waiting for user intervention? I would avoid issuing or causing
> a SIGTERM.
I have a use case where I would like to silently *ignore* something one
of my servers is sending me (one route that is sent conflicts with local
stuff, but that is not malicious or neglicience on their end, but something
on my end). So v1 would do that for me, changing it to "you can only
have accept ore reject-the-server" would make this option not work for
me (right now I use route-nopull and manually set route statements).
But anyway - I think SIGHUP is good enough. If there is no other server,
it will loop, like in any other uncorrectable error (like, server cert
invalid), but there is not much we can do about it.
> > We briefly discussed this yesterday - 64 should be plenty for about
> > everything, but a dynamic list might be the way to cope for the *one*
> > user who then comes up with the 65th rule...
>
> I didn't make it a linked-list mainly because of the need of adding to the
> tail of the list which means some extra extra book-keeping and if-elses.
Was more thinking about realloc()ing the array, not linked lists - but
let's keep it static for now, and just tell people "your config is too
big".
gert
--
USENET is *not* the non-clickable part of WWW!
//www.muc.de/~gert/
Gert Doering - Munich, Germany [email protected]
fax: +49-89-35655025 [email protected]
signature.asc
Description: PGP signature
