Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent stack overflow when attachment update is run in Rails 6 #912

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

evgeniradev
Copy link

Summary

To replicate this bug:
Use: gem 'rails', '~> 6.0.2', '>= 6.0.2.2'

Steps:

  1. Go to admin area
  2. Go to Files
  3. Upload a new file
  4. Find the newly created file on the Files' list and click on its Edit button
  5. In the file edit view, click Browse to select a new file from your system and click on the 'Update File' button

You might need to repeat step 5 twice to replicate this bug.

This PR contains a fix for this in case more people are facing this issue.

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