~juju/juju-core/trunk

Viewing all changes in revision 1274.

  • Committer: William Reade
  • Author(s): William Reade
  • Date: 2013-06-12 00:29:51 UTC
  • mfrom: (1262.4.4 juju-core)
  • Revision ID: fwereade@gmail.com-20130612002951-9abmjfet491duxci
state: Unit.WatchConfigSettings

state.Settings is bad, because it allows any client the ability to
read/change the associated document without any sort of validation;
and sending them down a channel is independently bad because the
documents don't contain the right data for the unit context.

So, I just made it an EntityWatcher instead. Works just as well, is
rather simpler, doesn't expose massive potential bugs. Win!

R=thumper, wallyworld, rog
CC=
https://codereview.appspot.com/10180043

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: