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

Digest::SHA1 is needed but not required #2

Closed
jlecour opened this issue Apr 25, 2009 · 1 comment
Closed

Digest::SHA1 is needed but not required #2

jlecour opened this issue Apr 25, 2009 · 1 comment

Comments

@jlecour
Copy link

jlecour commented Apr 25, 2009

Hi,

I can see tha Rack::Bug is using Digest::SHA1 but it's never explicitly required. On some of my apps, it's not required by any other plugin/gem so it's not loaded and I get this :

Status: 500 Internal Server Error
uninitialized constant Digest::SHA1

@jlecour
Copy link
Author

jlecour commented Apr 25, 2009

I've just added the require in the main lib and it's working :

diff --git a/vendor/plugins/rack-bug/lib/rack/bug.rb b/vendor/plugins/rack-bug/lib/rack/bug.rb
index ea36ebe..24128d6 100644
--- a/vendor/plugins/rack-bug/lib/rack/bug.rb
+++ b/vendor/plugins/rack-bug/lib/rack/bug.rb
@@ -1,4 +1,5 @@
 require "rack"
+require "digest/sha1"

 module Rack::Bug
   require "rack/bug/toolbar"

halorgium pushed a commit to halorgium/rack-bug that referenced this issue Dec 10, 2011
oggy pushed a commit to oggy/rack-bug that referenced this issue Sep 13, 2012
This issue was closed.
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

No branches or pull requests

1 participant