axsuul / rails-reverse-proxy

A reverse proxy for Ruby on Rails
MIT License
241 stars 61 forks source link

Remove rails dependency #45

Closed mediafinger closed 3 years ago

mediafinger commented 3 years ago

Apart from replacing the unnecessary heavy rails dependency with actionpack, I've also removed the unused rack-proxy dependency. rack is now no longer listed as an explicit dependency, but it is implicitly included over actionpack.

Not sure why this gem has a Gemfile.

As this gem has no test suite 😿 I bundled this updated version in our (closed source) app and ensured it still works as expected.

axsuul commented 3 years ago

Thanks @mediafinger for jumping on this!

Yep, a test suite is much needed but appreciate you testing it locally on your end. Looks good to me 👍

axsuul commented 3 years ago

@mediafinger Please use 0.10.1 release, thanks again!

mediafinger commented 3 years ago

Thank you for merging and releasing so quickly!