Skip to content

Commit f973075

Browse files
authored
Drop default permissions policy initializer (rails#52341)
It is too rarely used to warrant such a prominent placement in the default structure.
1 parent 2f92b1c commit f973075

File tree

8 files changed

+0
-62
lines changed

8 files changed

+0
-62
lines changed

actionmailbox/test/dummy/config/initializers/permissions_policy.rb

Lines changed: 0 additions & 13 deletions
This file was deleted.

actiontext/test/dummy/config/initializers/permissions_policy.rb

Lines changed: 0 additions & 13 deletions
This file was deleted.

activestorage/test/dummy/config/initializers/permissions_policy.rb

Lines changed: 0 additions & 13 deletions
This file was deleted.

railties/lib/rails/generators/rails/app/app_generator.rb

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,6 @@ def config_when_updating
145145
asset_manifest_exist = File.exist?("app/assets/config/manifest.js")
146146
asset_app_stylesheet_exist = File.exist?("app/assets/stylesheets/application.css")
147147
csp_config_exist = File.exist?("config/initializers/content_security_policy.rb")
148-
permissions_policy_config_exist = File.exist?("config/initializers/permissions_policy.rb")
149148

150149
@config_target_version = Rails.application.config.loaded_config_version || "5.0"
151150

@@ -179,10 +178,6 @@ def config_when_updating
179178
unless csp_config_exist
180179
remove_file "config/initializers/content_security_policy.rb"
181180
end
182-
183-
unless permissions_policy_config_exist
184-
remove_file "config/initializers/permissions_policy.rb"
185-
end
186181
end
187182
end
188183

@@ -559,7 +554,6 @@ def delete_action_cable_files_skipping_action_cable
559554
def delete_non_api_initializers_if_api_option
560555
if options[:api]
561556
remove_file "config/initializers/content_security_policy.rb"
562-
remove_file "config/initializers/permissions_policy.rb"
563557
end
564558
end
565559

railties/lib/rails/generators/rails/app/templates/config/initializers/permissions_policy.rb.tt

Lines changed: 0 additions & 13 deletions
This file was deleted.

railties/test/generators/api_app_generator_test.rb

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,6 @@ def test_app_update_does_not_generate_unnecessary_config_files
115115

116116
assert_no_file "config/initializers/assets.rb"
117117
assert_no_file "config/initializers/content_security_policy.rb"
118-
assert_no_file "config/initializers/permissions_policy.rb"
119118
end
120119

121120
def test_app_update_does_not_generate_unnecessary_bin_files
@@ -187,7 +186,6 @@ def skipped_files
187186
bin/yarn
188187
config/initializers/assets.rb
189188
config/initializers/content_security_policy.rb
190-
config/initializers/permissions_policy.rb
191189
lib/assets
192190
test/helpers
193191
public/404.html

railties/test/generators/app_generator_test.rb

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,6 @@
5353
config/initializers/content_security_policy.rb
5454
config/initializers/filter_parameter_logging.rb
5555
config/initializers/inflections.rb
56-
config/initializers/permissions_policy.rb
5756
config/locales/en.yml
5857
config/master.key
5958
config/puma.rb

railties/test/generators/plugin_generator_test.rb

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,6 @@
5656
test/dummy/config/initializers/content_security_policy.rb
5757
test/dummy/config/initializers/filter_parameter_logging.rb
5858
test/dummy/config/initializers/inflections.rb
59-
test/dummy/config/initializers/permissions_policy.rb
6059
test/dummy/config/locales/en.yml
6160
test/dummy/config/puma.rb
6261
test/dummy/config/routes.rb

0 commit comments

Comments
 (0)