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

Depends on ActiveSupport #1

Open
rkh opened this issue May 31, 2010 · 2 comments
Open

Depends on ActiveSupport #1

rkh opened this issue May 31, 2010 · 2 comments

Comments

@rkh
Copy link

rkh commented May 31, 2010

Good work. For a seemingly slim middleware depending on AS is rather heavy, imho. I was thinking about adding it to the BigBand stack, but depending on AS is a no go. I understand that you might not consider this change, just wanted to let you know.

@hassox
Copy link
Owner

hassox commented May 31, 2010

ActiveSupport is very limited in Rack::Rescue. It only loads what is needed, inheritable accessors string methods and such. It's deliberately kept as small as possible.

@rkh
Copy link
Author

rkh commented May 31, 2010

Yes, already saw that. I just had someone complain about even having to install AS and potentially loading two different extension libs. Hmmm... I'll think about that.

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

2 participants