search

wireframe / multitenant

making cross tenant data leaks a thing of the past.
http://blog.codecrate.com/2011/03/multitenant-locking-down-your-app-and.html
MIT License
164 stars 36 forks source link

unexpected-return-localjumperror fix #12

Closed racx closed 9 years ago

wireframe commented 10 years ago

can you provide any details on the error you encountered? what version of ruby/rails/etc? stacktrace?

tacticiankerala commented 9 years ago

@wireframe , I am using ruby 2.1 and Rails 4.1. The same error is happening to me.

Please merge the fix.

wireframe commented 9 years ago

@tacticiankerala reproducing the issue was required before releasing this patch so I've setup continuous integration for this project and verified the issue under ruby 2.1.

This patch has been merged and version 0.4.1 of the gem has been released with this fix.

Thank you @racx for your contribution!

tacticiankerala commented 9 years ago

Can we update the gem version as well? This is gem is a dependency for the spree_multi_tenant gem. So I can just bump up the version there for the latest changes. On Dec 19, 2014 9:43 PM, "Ryan Sonnek" notifications@github.com wrote:

@tacticiankerala https://github.com/tacticiankerala reproducing the issue was required before releasing this patch so I've setup continuous integration for this project and verified the issue under ruby 2.1.

This patch has been merged and version 0.4.1 of the gem has been released with this fix.

Thank you @racx https://github.com/racx for your contribution!

— Reply to this email directly or view it on GitHub https://github.com/wireframe/multitenant/pull/12#issuecomment-67658512.

wireframe commented 9 years ago

@tacticiankerala see previous comment. #13 was released which bumped the gem version to 0.4.1.

tacticiankerala commented 9 years ago

I am sorry. Just ignore me. I didn't see the complete message. Thanks a ton for taking action on it in no time. It was very critical for me tomorrow :). On Dec 19, 2014 9:53 PM, "Sreenath.N" tacticiankerala@gmail.com wrote:

Can we update the gem version as well? This is gem is a dependency for the spree_multi_tenant gem. So I can just bump up the version there for the latest changes. On Dec 19, 2014 9:43 PM, "Ryan Sonnek" notifications@github.com wrote:

@tacticiankerala https://github.com/tacticiankerala reproducing the issue was required before releasing this patch so I've setup continuous integration for this project and verified the issue under ruby 2.1.

This patch has been merged and version 0.4.1 of the gem has been released with this fix.

Thank you @racx https://github.com/racx for your contribution!

— Reply to this email directly or view it on GitHub https://github.com/wireframe/multitenant/pull/12#issuecomment-67658512.