Always reload token store on config reload; only set when successful
Review Request #10011 — Created June 11, 2018 and submitted — Latest diff uploaded
We now always reload the token store on a config reload to make our
behaviour identical to that of the htpasswd store. This allows
administrators to revoke tokens and trigger a reload without restarting
the server.Additionally, we were setting the token store early in
setConfigUnsafe
, which resulted in a partial state update if there
was an error later. We now make sure to update either everything or
nothing.Testing done:
Rango fmt ./...
.
Rango test ./...
.
Ran
go fmt ./...
.
Rango test ./...
.