Skip to content

Commit dbd5d1f

Browse files
G-RathG-Rath
committed
fix: write a safer gsub_file! 
1 parent 5a59ebc commit dbd5d1f

File tree

17 files changed

+142
-134
lines changed

17 files changed

+142
-134
lines changed

‎template.rb

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -381,4 +381,12 @@ def create_initial_migration
381381
run_with_clean_bundler_env "bin/rake db:migrate"
382382
end
383383

384+
def gsub_file!(path, flag, *args, &block)
385+
content = File.binread(path)
386+
387+
gsub_file(path, flag, *args, &block)
388+
389+
raise StandardError, "the contents of #{path} did not change!" if content == File.binread(path)
390+
end
391+
384392
apply_template!

‎variants/backend-base/app/template.rb

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,6 @@
1717
empty_directory_with_keep_file "app/services"
1818

1919
# Configure the default mailer to use the our default from address
20-
gsub_file "app/mailers/application_mailer.rb",
21-
"default from: '[email protected]'",
22-
"default from: Rails.application.config.app.mail_from"
20+
gsub_file! "app/mailers/application_mailer.rb",
21+
"default from: '[email protected]'",
22+
"default from: Rails.application.config.app.mail_from"

‎variants/backend-base/config/application.rb

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
gsub_file "config/application.rb",
2-
"# config.time_zone = 'Central Time (US & Canada)'",
3-
"config.time_zone = 'Wellington'"
1+
gsub_file! "config/application.rb",
2+
"# config.time_zone = 'Central Time (US & Canada)'",
3+
"config.time_zone = 'Wellington'"
44

55
insert_into_file "config/application.rb", after: /^require_relative ['"]boot['"]/ do
66
# the empty line at the beginning of this string is required

‎variants/backend-base/config/environments/development.rb

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,10 @@
1515
RUBY
1616
end
1717

18-
gsub_file "config/environments/development.rb",
19-
"join('tmp', 'caching-dev.txt')",
20-
'join("tmp/caching-dev.txt")'
18+
gsub_file! "config/environments/development.rb",
19+
"join('tmp', 'caching-dev.txt')",
20+
'join("tmp/caching-dev.txt")'
2121

22-
gsub_file "config/environments/development.rb",
23-
"config.action_controller.raise_on_missing_callback_actions = true",
24-
"# config.action_controller.raise_on_missing_callback_actions = true"
22+
gsub_file! "config/environments/development.rb",
23+
"config.action_controller.raise_on_missing_callback_actions = true",
24+
"# config.action_controller.raise_on_missing_callback_actions = true"

‎variants/backend-base/config/environments/production.rb

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -8,14 +8,14 @@
88
RUBY
99
end
1010

11-
gsub_file "config/environments/production.rb",
12-
"# config.force_ssl = true",
13-
<<~RUBY
14-
##
15-
# `force_ssl` defaults to on. Set `force_ssl` to false if (and only if) RAILS_FORCE_SSL=false, otherwise set it to true.
16-
#
17-
config.force_ssl = ENV.fetch("RAILS_FORCE_SSL", "true").downcase != "false"
18-
RUBY
11+
gsub_file! "config/environments/production.rb",
12+
"# config.force_ssl = true",
13+
<<~RUBY
14+
##
15+
# `force_ssl` defaults to on. Set `force_ssl` to false if (and only if) RAILS_FORCE_SSL=false, otherwise set it to true.
16+
#
17+
config.force_ssl = ENV.fetch("RAILS_FORCE_SSL", "true").downcase != "false"
18+
RUBY
1919

2020
insert_into_file "config/environments/production.rb",
2121
after: /# config\.action_mailer\.raise_deliv.*\n/ do
@@ -42,13 +42,13 @@
4242
RUBY
4343
end
4444

45-
gsub_file "config/environments/production.rb",
46-
"config.log_level = :info",
47-
'config.log_level = ENV.fetch("LOG_LEVEL", "info").to_sym'
45+
gsub_file! "config/environments/production.rb",
46+
"config.log_level = :info",
47+
'config.log_level = ENV.fetch("LOG_LEVEL", "info").to_sym'
4848

49-
gsub_file "config/environments/production.rb",
50-
"ActiveSupport::Logger.new(STDOUT)",
51-
"ActiveSupport::Logger.new($stdout)"
49+
gsub_file! "config/environments/production.rb",
50+
"ActiveSupport::Logger.new(STDOUT)",
51+
"ActiveSupport::Logger.new($stdout)"
5252

5353
insert_into_file "config/environments/production.rb",
5454
after: /.*config\.public_file_server\.enabled.*\n/ do

‎variants/backend-base/config/environments/test.rb

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
gsub_file "config/environments/test.rb",
2-
"config.eager_load = false",
3-
"config.eager_load = defined?(SimpleCov).present?"
1+
gsub_file! "config/environments/test.rb",
2+
"config.eager_load = false",
3+
"config.eager_load = defined?(SimpleCov).present?"
44

55
insert_into_file \
66
"config/environments/test.rb",
@@ -17,6 +17,6 @@
1717
RUBY
1818
end
1919

20-
gsub_file "config/environments/test.rb",
21-
"config.action_controller.raise_on_missing_callback_actions = true",
22-
"# config.action_controller.raise_on_missing_callback_actions = true"
20+
gsub_file! "config/environments/test.rb",
21+
"config.action_controller.raise_on_missing_callback_actions = true",
22+
"# config.action_controller.raise_on_missing_callback_actions = true"

‎variants/backend-base/config/template.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
copy_file "variants/backend-base/config/initializers/check_env.rb", "config/initializers/check_env.rb"
1818
copy_file "variants/backend-base/config/initializers/sentry.rb", "config/initializers/sentry.rb"
1919

20-
gsub_file "config/initializers/filter_parameter_logging.rb", /\[:password\]/ do
20+
gsub_file! "config/initializers/filter_parameter_logging.rb", /\[:password\]/ do
2121
"%w[password secret session cookie csrf]"
2222
end
2323

@@ -59,7 +59,7 @@
5959
EO_ROUTES
6060

6161
if File.exist? "config/storage.yml"
62-
gsub_file "config/storage.yml", /# service: S3/ do
62+
gsub_file! "config/storage.yml", /# service: S3/ do
6363
<<~YAML
6464
# service: S3
6565
# upload:

‎variants/deploy_with_ackama_ec2_capistrano/template.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -177,7 +177,7 @@
177177
178178
EO_RUBY
179179

180-
gsub_file("config/deploy.rb", old_generated_cap_config_snippet, new_ackama_cap_config_snippet)
180+
gsub_file!("config/deploy.rb", old_generated_cap_config_snippet, new_ackama_cap_config_snippet)
181181

182182
insert_into_file "Capfile", after: /install_plugin Capistrano::SCM::Git/ do
183183
<<~EO_RUBY

‎variants/deploy_with_capistrano/template.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,7 @@
165165
166166
EO_RUBY
167167

168-
gsub_file("config/deploy.rb", old_generated_cap_config_snippet, new_ackama_cap_config_snippet)
168+
gsub_file!("config/deploy.rb", old_generated_cap_config_snippet, new_ackama_cap_config_snippet)
169169

170170
insert_into_file "Capfile", after: /install_plugin Capistrano::SCM::Git/ do
171171
<<~EO_RUBY

‎variants/devise/template.rb

Lines changed: 64 additions & 64 deletions
Original file line numberDiff line numberDiff line change
@@ -10,29 +10,29 @@
1010
TERMINAL.puts_header "Generating User model with devise"
1111
run "bundle exec rails generate devise User"
1212

13-
gsub_file "app/models/user.rb",
14-
":validatable",
15-
":validatable, :lockable"
13+
gsub_file! "app/models/user.rb",
14+
":validatable",
15+
":validatable, :lockable"
1616

1717
devise_migration_filename = Dir.children("db/migrate").find { |filename| filename.end_with?("_devise_create_users.rb") }
1818
devise_migration_path = "db/migrate/#{devise_migration_filename}"
1919

2020
TERMINAL.puts_header "Tweaking auto-generated devise migration '#{devise_migration_path}'"
21-
gsub_file devise_migration_path,
22-
" # t.integer :failed_attempts",
23-
" t.integer :failed_attempts"
24-
gsub_file devise_migration_path,
25-
" # t.string :unlock_token",
26-
" t.string :unlock_token"
27-
gsub_file devise_migration_path,
28-
" # t.datetime :locked_at",
29-
" t.datetime :locked_at"
30-
gsub_file devise_migration_path,
31-
" # add_index :users, :unlock_token",
32-
" add_index :users, :unlock_token"
33-
gsub_file devise_migration_path,
34-
/ # add_index :users, :unlock_token.+/,
35-
" add_index :users, :unlock_token, unique: true"
21+
gsub_file! devise_migration_path,
22+
" # t.integer :failed_attempts",
23+
" t.integer :failed_attempts"
24+
gsub_file! devise_migration_path,
25+
" # t.string :unlock_token",
26+
" t.string :unlock_token"
27+
gsub_file! devise_migration_path,
28+
" # t.datetime :locked_at",
29+
" t.datetime :locked_at"
30+
gsub_file! devise_migration_path,
31+
" # add_index :users, :unlock_token",
32+
" add_index :users, :unlock_token"
33+
gsub_file! devise_migration_path,
34+
/ # add_index :users, :unlock_token.+/,
35+
" add_index :users, :unlock_token, unique: true"
3636

3737
TERMINAL.puts_header "Running db migration"
3838
run "bundle exec rails db:migrate"
@@ -45,51 +45,51 @@
4545
#
4646
TERMINAL.puts_header "Tweaking config/initializers/devise.rb"
4747

48-
gsub_file "config/initializers/devise.rb",
49-
" config.mailer_sender = '[email protected]'",
50-
" config.mailer_sender = Rails.application.config.app.mail_from"
48+
gsub_file! "config/initializers/devise.rb",
49+
" config.mailer_sender = '[email protected]'",
50+
" config.mailer_sender = Rails.application.config.app.mail_from"
5151

52-
gsub_file "config/initializers/devise.rb",
53-
" # config.scoped_views = false",
54-
" config.scoped_views = true"
52+
gsub_file! "config/initializers/devise.rb",
53+
" # config.scoped_views = false",
54+
" config.scoped_views = true"
5555

56-
gsub_file "config/initializers/devise.rb",
57-
" config.password_length = 6..128",
58-
" config.password_length = 16..128"
56+
gsub_file! "config/initializers/devise.rb",
57+
" config.password_length = 6..128",
58+
" config.password_length = 16..128"
5959

60-
gsub_file "config/initializers/devise.rb",
61-
" # config.paranoid = true",
62-
" config.paranoid = true"
60+
gsub_file! "config/initializers/devise.rb",
61+
" # config.paranoid = true",
62+
" config.paranoid = true"
6363

64-
gsub_file "config/initializers/devise.rb",
65-
/ # config.secret_key = '.+'/,
66-
" # config.secret_key = 'do_not_put_secrets_in_source_control_please'"
64+
gsub_file! "config/initializers/devise.rb",
65+
/ # config.secret_key = '.+'/,
66+
" # config.secret_key = 'do_not_put_secrets_in_source_control_please'"
6767

68-
gsub_file "config/initializers/devise.rb",
69-
/ # config.lock_strategy = .+/,
70-
" config.lock_strategy = :failed_attempts"
68+
gsub_file! "config/initializers/devise.rb",
69+
/ # config.lock_strategy = .+/,
70+
" config.lock_strategy = :failed_attempts"
7171

72-
gsub_file "config/initializers/devise.rb",
73-
/ # config.unlock_strategy = .+/,
74-
" config.unlock_strategy = :email"
72+
gsub_file! "config/initializers/devise.rb",
73+
/ # config.unlock_strategy = .+/,
74+
" config.unlock_strategy = :email"
7575

76-
gsub_file "config/initializers/devise.rb",
77-
" # config.parent_mailer = 'ActionMailer::Base'",
78-
" config.parent_mailer = 'ApplicationMailer'"
76+
gsub_file! "config/initializers/devise.rb",
77+
" # config.parent_mailer = 'ActionMailer::Base'",
78+
" config.parent_mailer = 'ApplicationMailer'"
7979

80-
gsub_file "config/initializers/devise.rb",
81-
/ # config.maximum_attempts = .+/,
82-
<<-EO_CHUNK
80+
gsub_file! "config/initializers/devise.rb",
81+
/ # config.maximum_attempts = .+/,
82+
<<-EO_CHUNK
8383
#
8484
# https://www.nzism.gcsb.govt.nz/ism-document/#1887 recommends 3 as a default. FYI to
8585
# be fully compliant with https://www.nzism.gcsb.govt.nz/ism-document/#1887 then only
8686
# Administrators should be able to unlock.
8787
config.maximum_attempts = 3
88-
EO_CHUNK
88+
EO_CHUNK
8989

90-
gsub_file "config/initializers/devise.rb",
91-
/ # config.last_attempt_warning = .+/,
92-
" config.last_attempt_warning = true"
90+
gsub_file! "config/initializers/devise.rb",
91+
/ # config.last_attempt_warning = .+/,
92+
" config.last_attempt_warning = true"
9393

9494
##
9595
# Add a block to config/routes.rb demonstrating how to create authenticated
@@ -140,13 +140,13 @@
140140

141141
copy_file "app/controllers/users/sessions_controller.rb"
142142

143-
gsub_file "config/routes.rb",
144-
"devise_for :users",
145-
<<~EO_DEVISE
146-
devise_for :users, controllers: {
147-
sessions: "users/sessions"
148-
}
149-
EO_DEVISE
143+
gsub_file! "config/routes.rb",
144+
"devise_for :users",
145+
<<~EO_DEVISE
146+
devise_for :users, controllers: {
147+
sessions: "users/sessions"
148+
}
149+
EO_DEVISE
150150

151151
insert_into_file "app/models/user.rb", before: /^end/ do
152152
<<~'RUBY'
@@ -202,14 +202,14 @@ def authenticatable_salt
202202
copy_file "spec/requests/session_cookie_expiry_spec.rb"
203203

204204
# tell pundit not to check that authorization was called on devise controllers
205-
gsub_file("app/controllers/application_controller.rb",
206-
"after_action :verify_authorized, except: :index",
207-
"after_action :verify_authorized, except: :index, unless: :devise_controller?"
208-
)
209-
gsub_file("app/controllers/application_controller.rb",
210-
"after_action :verify_policy_scoped, only: :index",
211-
"after_action :verify_policy_scoped, only: :index, unless: :devise_controller?"
212-
)
205+
gsub_file!("app/controllers/application_controller.rb",
206+
"after_action :verify_authorized, except: :index",
207+
"after_action :verify_authorized, except: :index, unless: :devise_controller?"
208+
)
209+
gsub_file!("app/controllers/application_controller.rb",
210+
"after_action :verify_policy_scoped, only: :index",
211+
"after_action :verify_policy_scoped, only: :index, unless: :devise_controller?"
212+
)
213213

214214
TERMINAL.puts_header "Running rubocop -A to fix formatting in files generated by devise"
215215
run "bundle exec rubocop -A -c ./.rubocop.yml"

‎variants/frontend-base-typescript/template.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ def rename_js_file_to_ts(file)
3535
copy_file ".eslintrc.js", force: true
3636
copy_file "types.d.ts", force: true
3737

38-
gsub_file(
38+
gsub_file!(
3939
"app/frontend/packs/application.ts",
4040
"process.env.SENTRY_ENV || process.env.RAILS_ENV",
4141
"process.env.SENTRY_ENV ?? process.env.RAILS_ENV"
@@ -53,7 +53,7 @@ def rename_js_file_to_ts(file)
5353
return String(images(name));
5454
};
5555
EO_TS_ENABLE_IMAGES
56-
gsub_file("app/frontend/packs/application.ts", js_load_images_chunk, ts_load_images_chunk, force: true)
56+
gsub_file!("app/frontend/packs/application.ts", js_load_images_chunk, ts_load_images_chunk, force: true)
5757

5858
update_package_json do |package_json|
5959
package_json["scripts"]["typecheck"] = "tsc -p . --noEmit"

‎variants/frontend-base/sentry/template.rb

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,6 @@
4747
EO_JS
4848
end
4949

50-
gsub_file "config/initializers/content_security_policy.rb",
51-
/# policy.report_uri ".+"/,
52-
'policy.report_uri(ENV["SENTRY_CSP_HEADER_REPORT_ENDPOINT"]) if ENV["SENTRY_CSP_HEADER_REPORT_ENDPOINT"]'
50+
gsub_file! "config/initializers/content_security_policy.rb",
51+
/# policy.report_uri ".+"/,
52+
'policy.report_uri(ENV["SENTRY_CSP_HEADER_REPORT_ENDPOINT"]) if ENV["SENTRY_CSP_HEADER_REPORT_ENDPOINT"]'

0 commit comments

Comments
 (0)