<p dir="ltr">It feels like this should be something that the API server is looking up in the database/some sort of query, not just munging a URL. I'm fine with having a Params struct, though I worry that it isn't really appropriate to be passing all possible Params to all facades. They already have a State connection, I'd like us to at least be designing the layering appropriately.  I feel like state.State is intended to be where "Knowledge" resides. Why isn't what we want there?<br>
</p>
<p dir="ltr">John<br>
=:-></p>
<div class="gmail_quote">On Sep 3, 2014 6:03 PM, "Andrew Wilkins" <<a href="mailto:andrew.wilkins@canonical.com">andrew.wilkins@canonical.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">I'm working on moving all tools downloads to download from the API server. There will are a few APIs that return tools:<div>  - Upgrader.Tools</div><div>  - Client.FindTools</div><div>  - Provisioner.FindTools</div>

<div><br></div><div>These APIs will need to return URLs pointing at the API server. I'm intending to change the facade constructors to take a parameter struct, and extend it with the API server's root URL (i.e. https://<address>:<port>/environment/<uuid>), where the address is the one used by the client to connect.</div>

<div><br></div><div>Any reason I should not go ahead and do that? This will probably make it easier to slot in a "Restarter" or whatever as well.</div><div><br></div><div>Cheers,</div><div>Andrew</div></div>
<br>--<br>
Juju-dev mailing list<br>
<a href="mailto:Juju-dev@lists.ubuntu.com">Juju-dev@lists.ubuntu.com</a><br>
Modify settings or unsubscribe at: <a href="https://lists.ubuntu.com/mailman/listinfo/juju-dev" target="_blank">https://lists.ubuntu.com/mailman/listinfo/juju-dev</a><br>
<br></blockquote></div>