Skip to content

Conversation

@iamvery
Copy link
Contributor

@iamvery iamvery commented Nov 20, 2013

Here's an alternate approach to #25 that provides a little more flexibility. Now individual options may be provided with the --ssh-options cli option. For convenience, I also added the --no-forwarding option that adds All The Things.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps these options should be stored in a constant rather than literals here?

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling c5b4cac on iamvery:ssl-options into 4befc00 on chrishunt:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 85344c4 on iamvery:ssl-options into 4befc00 on chrishunt:master.

@iamvery
Copy link
Contributor Author

iamvery commented Nov 20, 2013

Lol, travis. I just can't make you happy 😄

@iamvery
Copy link
Contributor Author

iamvery commented Nov 21, 2013

Just noticed the name of this branch is "ssl-options"... yeah, long day

iamvery added a commit to iamvery/dotfiles that referenced this pull request Nov 21, 2013
* This option would disable forwarding for the pair's SSH connection.
* See chrishunt/github-auth#26 for more
  information.
@iamvery
Copy link
Contributor Author

iamvery commented May 23, 2014

@chrishunt these have gotten so stale. Is this something you want to continue to think about, or should I just close them out and forget it? I'm happy to fix up the PRs if you'd like me to

@chrishunt
Copy link
Owner

@iamvery Are you OK with closing this out for now? Sorry I dropped the ball on it.

I think it will make sense to re-open in the future if enough people show interest. Right now, I'm not sure there is a lot of pressure to get this in.

@iamvery
Copy link
Contributor Author

iamvery commented Jun 16, 2014

@chrishunt yeah, no worries from me. This is something that I continue to want, but I understand if the demand isn't there 😸

@iamvery iamvery closed this Jun 16, 2014
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

Successfully merging this pull request may close these issues.

3 participants