DEV: Remove staff bypass on fix missing

After thinking about it, I worry that this will potentially leave a site
setting set when people hit ctrl-c ... feels a tiny bit risky, so leaving
it out.
This commit is contained in:
Sam Saffron 2020-08-28 12:35:35 +10:00
parent ed9323043f
commit 80e17f5f9e
No known key found for this signature in database
GPG Key ID: B9606168D2FFD9F5
1 changed files with 2 additions and 10 deletions

View File

@ -1181,20 +1181,12 @@ def fix_missing_s3
puts
end
def allow_all_uploads
old_staff_extension = SiteSetting.authorized_extensions_for_staff
SiteSetting.authorized_extensions_for_staff = "*"
yield
ensure
SiteSetting.authorized_extensions_for_staff = old_staff_extension
end
task "uploads:fix_missing_s3" => :environment do
if RailsMultisite::ConnectionManagement.current_db != "default"
allow_all_uploads { fix_missing_s3 }
fix_missing_s3
else
RailsMultisite::ConnectionManagement.each_connection do
allow_all_uploads { fix_missing_s3 }
fix_missing_s3
end
end
end