build(deps): bump activestorage from 8.1.3 to 8.1.3.1 - #1018
build(deps): bump activestorage from 8.1.3 to 8.1.3.1#1018dependabot[bot] wants to merge 1 commit into
Conversation
Bumps [activestorage](https://github.com/rails/rails) from 8.1.3 to 8.1.3.1. - [Release notes](https://github.com/rails/rails/releases) - [Changelog](https://github.com/rails/rails/blob/v8.1.3.1/activestorage/CHANGELOG.md) - [Commits](rails/rails@v8.1.3...v8.1.3.1) --- updated-dependencies: - dependency-name: activestorage dependency-version: 8.1.3.1 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.
TIP This summary will be updated as you push new changes.
There was a problem hiding this comment.
Pull Request Overview
While Codacy indicates the changes are up to standards, there is an implementation gap: the PR does not update config.active_storage.variable_content_types as recommended by the security advisory for CVE-2026-66066. This omission could lead to unhandled exceptions when the application processes untrusted image formats. Furthermore, ensure that all deployment environments have compatible system libraries, as the updated dependency will trigger a RuntimeError at boot if the underlying requirements are not met.
About this PR
- The security advisory recommends updating
config.active_storage.variable_content_typesto exclude blocked formats. This configuration change is missing from the current PR and should be added to ensure the application correctly handles the security restrictions. - The application will now raise a
RuntimeErrorat boot if the system libvips or ruby-vips dependencies do not meet the new minimum requirements. Ensure CI and production environments are verified for compatibility before merging. - This update introduces a breaking change for applications processing specific image types such as BMP, ICO, PSD, SVG, and JPEG XL. These formats are now blocked by default, which may result in exceptions or missing metadata during processing.
Test suggestions
- Verify application boots successfully with the new Active Storage initialization logic.
- Verify standard image variants (PNG, JPEG) still generate correctly.
- Verify behavior for 'untrusted' formats like BMP, PSD, or ICO (expecting Vips::Error or missing metadata).
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Verify application boots successfully with the new Active Storage initialization logic.
2. Verify standard image variants (PNG, JPEG) still generate correctly.
3. Verify behavior for 'untrusted' formats like BMP, PSD, or ICO (expecting Vips::Error or missing metadata).
TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback
Bumps activestorage from 8.1.3 to 8.1.3.1.
Release notes
Sourced from activestorage's releases.
... (truncated)
Changelog
Sourced from activestorage's changelog.
Commits
3989ebfPreparing for 8.1.3.1 release349e7a5Disable libvips's unfuzzed image loaders and saversDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)You can disable automated security fix PRs for this repo from the Security Alerts page.