<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>...</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

>From what I can tell poking around the code base, the only place that still uses the hash(password) is actually in the Dummy provider.<br>
<br>
</blockquote>
<br></div>
Right, and when I remove that code all the tests pass with some session copying in place!<br>
<br>
<a href="https://github.com/voidspace/juju/compare/master...copy-sessions" target="_blank">https://github.com/voidspace/<u></u>juju/compare/master...copy-<u></u>sessions</a><div class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I feel like we're at a point where we can safely remove that from the Dummy provider, and also remove the fallback code in our 'connect to the database' code. (If we leave it in, then I think after<br>
</blockquote></div>
Do you mean the "oldPassword" logic in cmd/jujud/agent.go (I had to add code there to re-open the state when we change the password.)<br></blockquote><div><br></div><div>What I mean is removing the "if IsUnauthorized(err) { altpassword = hash(password); login(alt password); SetPassword(password) }"</div>
<div>Because *if* the current access is actually hash(password) we still (?) need to set it to the real password. (and then reconnect).</div><div><br></div><div>I'd be ok removing the fallback, as I don't think there will be any real sites out there that need it in production. Those that might have would have already run the fallback code.</div>
<div><br></div><div>John</div><div>=:-></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
All the best,<br>
<br>
Michael<div class="HOEnZb"><div class="h5"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
changing the password just reconnecting to the database is fine, because it should happen infrequently.<br>
<br>
Thoughts?<br>
<br>
John<br>
=:-><br>
<br>
</blockquote>
<br>
</div></div></blockquote></div><br></div></div>