mod_pubsub: Update callbacks on reload to more completely refresh config This would also release any resources held via references from these callbacks. I'm not sure why we don't just re-new() the whole thing. Related to #1382

migrator: Silence assert in core.moduleapi The assert triggers because we're not loading the stanza route, because we are unlikely to need it during migration.

migrator: Use parsed command line flags already parsed by util.startup

migrator: Trick net.server into thinking the config is loaded Fixes "Loading outside Prosody or Prosody not yet initialized"

migrator: Customise startup sequence to fix #1673 (Thanks acidsys) Diverge from util.startup.prosodyctl() in order to skip unneeded behavior, such as loading the *Prosody* config file, which we do not need here, based on the `--config` flag which should point at the migrator config file instead. Notably removed: * read_config() since this loads the Prosody config * check_unwriteable() which checks logfiles specified in the Prosody config, so not relevant * make_dummy_hosts() but the migrator sets up its own hosts during migration

Fix various spelling errors (thanks codespell) Also special thanks to timeless, for wordlessly reminding me to check for typos.

mod_pubsub: Update configuration on reload (fixes #1382) Because of the way the previous pubsub service is carried access module reloads, it would retain the configuration options with their previous values from when the service was first created.

mod_pubsub: Silence warning about 'service' as argument [luacheck] Maybe the 'service' in the outer scope should be moved down to minimize overlap with other functions that receive the same service as argument?

mod_pep: Remove now unused util.cache import

util.pubsub: Signal that 'persistent-items' is unsupported when disabled XEP-0060 says that this the way to indicate that 'persistent-items' is unsupported, but doesn't explicitly say if it being disabled in the node configuration also counts as unsupported.