this function does a storage->save() which is a hideously expensive operation (because it causes a dataChanged signal to be emitted which forces all clients connected to reload all data).  Is this function called once for every request, or once altogether, or what?  If it's called more than once per sync cycle, we should fix that (in the future).