Skip to content

Bump rails to 8.1.x + HEALTHCHECK - #327

Open
3kh0 wants to merge 6 commits into
mainfrom
rails-8.1
Open

3kh0 wants to merge 6 commits into
mainfrom
rails-8.1

Conversation

@3kh0

@3kh0 3kh0 commented Sep 6, 2026

Copy link
Copy Markdown
Member

This pull

  • Brings us up to date on the latest rails security patch
  • Adds a HEALTHCHECK to the dockerfile so we don't shoot ourselves in the foot
  • Bumps smaller gems for security patches + parity with the recommended rails list

@socket-security

socket-security Bot commented Sep 6, 2026

Copy link
Copy Markdown

@socket-security

socket-security Bot commented Sep 6, 2026

Copy link
Copy Markdown

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Warn High
Obfuscated code: gem actionpack is 90.0% likely obfuscated

Confidence: 0.90

Location: Package overview

From: Gemfile.lockgem/rails_semantic_logger@4.19.0gem/vite_rails@3.0.20gem/public_activity@3.0.2gem/phlex-rails@2.4.0gem/web-console@4.3.0gem/rspec-rails@8.0.4gem/propshaft@1.3.2gem/good_job@4.19.2gem/rails@8.1.3.1gem/sentry-rails@6.7.0gem/doorkeeper@5.9.6gem/premailer-rails@1.12.0gem/wicked@2.0.0gem/letter_opener_web@3.0.0gem/audits1984@0.1.7gem/active_storage_encryption@0.3.0gem/factory_bot_rails@6.5.1gem/cloudflare-rails@7.0.0gem/actionpack@8.1.3.1

ℹ Read more on: This package | This alert | What is obfuscated code?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Packages should not obfuscate their code. Consider not using packages with obfuscated code.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore gem/actionpack@8.1.3.1. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

Warn High
Obfuscated code: gem activerecord is 90.0% likely obfuscated

Confidence: 0.90

Location: Package overview

From: Gemfile.lockgem/public_activity@3.0.2gem/flipper-active_record@1.4.2gem/good_job@4.19.2gem/annotaterb@4.24.0gem/rails@8.1.3.1gem/kaminari@1.2.2gem/hashid-rails@1.4.1gem/active_storage_encryption@0.3.0gem/paper_trail@17.0.0gem/acts_as_paranoid@0.11.0gem/activerecord@8.1.3.1

ℹ Read more on: This package | This alert | What is obfuscated code?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Packages should not obfuscate their code. Consider not using packages with obfuscated code.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore gem/activerecord@8.1.3.1. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

Warn High
Obfuscated code: gem activerecord is 90.0% likely obfuscated

Confidence: 0.90

Location: Package overview

From: Gemfile.lockgem/public_activity@3.0.2gem/flipper-active_record@1.4.2gem/good_job@4.19.2gem/annotaterb@4.24.0gem/rails@8.1.3.1gem/kaminari@1.2.2gem/hashid-rails@1.4.1gem/active_storage_encryption@0.3.0gem/paper_trail@17.0.0gem/acts_as_paranoid@0.11.0gem/activerecord@8.1.3.1

ℹ Read more on: This package | This alert | What is obfuscated code?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Packages should not obfuscate their code. Consider not using packages with obfuscated code.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore gem/activerecord@8.1.3.1. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

3kh0 added 3 commits September 6, 2026 13:17
Thruster listens on port 80. Without a HEALTHCHECK, Coolify/Traefik
treats the web container as having no available server.

Skip SSL and domain redirects for /up so the in-container probe
(curl http://127.0.0.1:80/up) gets 200 instead of 301.
@3kh0 3kh0 changed the title Bump rails to 8.1.x Bump rails to 8.1.x + HEALTHCHECK Sep 6, 2026
3kh0 added 3 commits September 6, 2026 13:34
Stay on the supported line for each gem rather than the newest major:

- puma 7.2.1 (not 8.0)
- sentry-ruby/rails 6.7.0 (not 7.0)
- doorkeeper 5.9.6 + openid_connect 1.10.5 (not 2.0 beta)
- good_job 4.19.2 with incremental index migrations
- image_processing stays 1.14 (Rails 8.1 template)

Also pick up current patches on the same majors (thruster, bootsnap,
propshaft, flipper, faraday, nokogiri, loofah, mail, json, erb).
Restore Alpine-driven templates that erb_lint autocorrect truncated, keep
schema.rb as a real dump (can_ban, login-code purpose, unique session
token index), and skip loopback instead of special-casing /up in
DomainRedirect and ssl_options.

Fail closed on missing Active Record encryption keys outside local,
leave 8.1 defaults opt-in via new_framework_defaults_8_1.rb, and DRY
the persona draft find-or-create lock path.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant