Re: [patch] proposed patch for managing timeouts on doveadm iteration with -A

>>>>> "John" == John Fawcett via dovecot <[email protected]> writes:

> having further investigated the timeouts mentioned in the previous 
> thread, I am submitting this patch as a potential solution.

I like this, it makes sense that the timeout is per-lookup, not
overall length.  Now I also admit that for 20,000 (or more!) look ups,
maybe you do want it to time out after a while... or give some
feedback on how far it's gotten into the lookup. 

> Basically with -A doveadm parameter, the 155 second timeout is
> applied to the whole end to end transaction. With -F parameter the
> 155 second timeout is applied to each single user
> operation. Changing the logic of -A to follow -F looks a bit
> invasive. So this patch does more or less the same thing,
> i.e. giving each user operation 155 seconds timeout, done in a
> different way. Whenever another username is returned, the timeout is
> reset.

Simple and clean patch.  

> PS I got help from Claude on this.

> best regards

> John




> _______________________________________________
> dovecot mailing list -- [email protected]
> To unsubscribe send an email to [email protected]
_______________________________________________
dovecot mailing list -- [email protected]
To unsubscribe send an email to [email protected]