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

change file image storage to memory on-the-fly #21

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

mrbrdo
Copy link

@mrbrdo mrbrdo commented May 14, 2012

change file image storage to memory on-the-fly

@jasonfb
Copy link

jasonfb commented Jun 8, 2012

does this make it so that this plug-in will work on Heroku's read-only file system?

@mrbrdo
Copy link
Author

mrbrdo commented Jun 8, 2012

It doesn't use any files (at least not for generating the image), so it might work yes.

@jasonfb
Copy link

jasonfb commented Jun 8, 2012

I'm getting this on Heroku when I try to get the image to load: uninitialized constant SimpleCaptcha::ImageHelpers::Magick
(although oddly the same configuration works for me locally)

the hiccup is in lib/simple_captcha/image.rb line 88, it is trying to call Magick

any ideas where Magick comes from?

I tried adding gem 'rmagick' to my application (Rails 3.0.9) gem file but that didn't quite work.

@mrbrdo
Copy link
Author

mrbrdo commented Jun 8, 2012

yeah you need rmagick... try adding to your Gemfile gem 'rmagick', :require => 'rmagick' and then bundle install

@jasonfb
Copy link

jasonfb commented Jun 8, 2012

yea I had the syntax wrong in my gem file, I found this and it seemed to fix that problem

gem "rmagick", "2.12.0", :require => 'RMagick'

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.

2 participants