Skip to content

Ability to skip Include directive and add smallstep config directly to ~/.ssh/config #246

Open

Description

@roopakv

What would you like to be added

Currently smallstep adds

Host *
	Include /path/to/smallstep/config

I would like a flag which rather does

Host *
	<Smallstep config here>
    Match exec "/usr/local/bin/step ssh check-host %h" 
		...

Why this is needed

We use IntelliJ IDEs which connect to dev servers, and IntelliJ uses sshJ which does not support the Include directive.

Activity

  1. dopey commented on Apr 7, 2020

    @dopey
    Contributor

    Great suggestion! Seems like we should definitely make this an option. We'll have to prioritize for one of our next sprints.

  2. added this to the Short Term Backlog milestone on Apr 15, 2020
  3. jhult commented on Jan 28, 2022

    @jhult

    @dopey, @sourishkrout, any news on this?

  4. dopey commented on Jan 28, 2022

    @dopey
    Contributor

    Hey @jhult only bad news, which is to say, no movement. If anything we've made this worse by adding another include indirection into that flow.

    I'll add this back to our triage so we can re-discuss it (fell off our map), but my gut feeling is that this will be kinda challenging / involved to special case. With the relatively recent introduction of contexts there is the possibility to have multiple match exec ... blocks. Managing those in one file is troublesome and it's the reason we've chose to use includes.

  5. added
    needs triageWaiting for discussion / prioritization by team
    on Jan 31, 2022
  6. dopey commented on Feb 3, 2022

    @dopey
    Contributor

    @jhult we had a chance to talk about this today, and unfortunately, this is going to be difficult for us to support "natively".

    There are two workarounds that we can think of:

    1. You can define your own template for ~/.ssh/config in the CA configuration. -- Or
    2. You can manage the ~/.ssh/config file using existing configuration mgmt solutions.

    If it were me, I'd probably do 2), but lmk if you'd like more info about replacing the default template.

    p.s. Templates attribute in the CA config: https://github.com/smallstep/certificates/blob/808f039b0993e047bff9302d42bb03bcc15e92ad/authority/config/config.go#L66

  7. removed
    needs triageWaiting for discussion / prioritization by team
    on Feb 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions