Skip to content

Conversation

@bradkwadsworth
Copy link

*Issue #5

Added flag to integrate with ssm. Also generates a temporary ssh_config file.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@bradkwadsworth bradkwadsworth requested review from ajorg-aws and amazon-auto and removed request for LordAlfredo February 7, 2020 15:30
@VikashKumarInd
Copy link

Thanks Brad bringing up this new feature change. We are reviewing the feature support and the change. We will get back to you when we make any decision.

@CptTZ CptTZ requested review from CptTZ and removed request for ajorg-aws and amazon-auto April 3, 2020 07:27
@wichert
Copy link

wichert commented Nov 23, 2020

@VikashKumarInd Can you give us a status update for this PR?

@elgalu
Copy link

elgalu commented Feb 11, 2021

any updates?

if region is not None:
region_cli = "--region {0}".format(region)

proxy_command = "ProxyCommand sh -c \"aws {0} ssm start-session --target %h --document-name AWS-StartSSHSession --parameters 'portNumber=%p'\"".format(region_cli)
Copy link
Contributor

Choose a reason for hiding this comment

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

Command should include the --profile argument if it was used when called mssh

Copy link
Contributor

Choose a reason for hiding this comment

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

Also it should adapt when executed on windows environments

ProxyCommand C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe "aws ssm start-session --target %h --document-name AWS-StartSSHSession --parameters portNumber=%p"

@Maks3w
Copy link
Contributor

Maks3w commented Feb 14, 2021

I've written an implementation works entirely inside of ssh config. No wrapper are needed.

https://gist.github.com/Maks3w/de72b0160d5e6af958adc42eab014624

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.

5 participants