Mon-Ouie / pry-remote

Connect to Pry remotely
Other
639 stars 81 forks source link

adding DRb::DRbUnknownError catching, adding client_port parameter #57

Closed yazgoo closed 8 years ago

yazgoo commented 10 years ago

Hi,

I'm currently using pry-remote for remote debugging on ruby 1.9 (more precisely jruby) over ssh tunneling. ( https://github.com/yazgoo/prysless/commit/999d1e7ea4584b3df2b29e48d99b8fa920732d2e ) What's missing is the ability to specify the port where the server should connect. So here is a pull request to add this client_port option, the default being the current (0). I also had an unrescued DRb::DRbUnknownError, on cleanup method at startup, so I catched it.

Thanks

Mon-Ouie commented 10 years ago

Hey,

Did that exception you got say anything? It isn't raised here and I'd rather not rescue exceptions that are actually bugs.

yazgoo commented 8 years ago

Closing this because it is too old.