Determining leader

[lucretius]

Determining leader

[Lucretius]

Hi @Kyrklund

My team swapped to using a Consul backend shortly after I had finished writing this so we don't actually run this in production. I think if we wanted to use isSelf as you pointed out we would need to call local Vault instance (and skip any TLS verification) of the individual nodes for the leader check, as invoking the Vault API address just calls into the HA cluster itself - which always returns isSelf of true.

I can't remember any reason why I did not opt for this approach, and assuming the existing behavior still works I don't have any problem changing it if it can help make the project usable for your team. Not sure when I would get to it, but if you have any interest in making a PR and testing out your changes, I would be happy to review and merge it in.

[Kyrklund]

I see, and totally understand :)
In our setup we do call the local address on each node, but we still use TLS (DNS record for each node with matching cert)
I did do PR just in case, and you can merge it in if you think it looks good
PR #4