Server selection by weight flawed?

Hello,

I think I've found a logic problem in the pen code when you're using
server selection by weight.

I noticed that when using weighted servers, after a server temporarily
died and came back, it wasn't used again.

If I'm following the code properly this appears to be because
try_server() is the function that marks servers as 'up' again, once
it's successfully connected, but server_by_weight() will never return
a broken server because of the status != 0 check. So broken servers
are never retested.

Am I reading that right?

-- 
Chris