DEV: Add ENV flag to restrict backups (PR #7945)

GitHub

You’ve signed the CLA, OsamaSayegh. Thank you! This pull request is ready for review.

Trivial but this could be one line if you like raise Discourse::InvalidAccess if BackupRestore.backups_disallowed?

Another trivial change but redis has an exists method which is slightly simpler.

Is this site setting the only way to trigger this? I would prefer if it could be passed as an option rather than setting a global setting like this.

I think this PR can be closed because Neil fixed the underlying problem this PR was meant to fix?

1 Like