-
Notifications
You must be signed in to change notification settings - Fork 9k
Add support for Include directive in SSH config parser (#19555) #19761
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Add support for Include directive in SSH config parser (#19555) #19761
Conversation
|
@AbhishekGiri04 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
|
Summary of the Pull Request
Adds support for the
Includedirective in SSH config parser, enabling Terminal to generate profiles from split SSH configuration files (e.g.,Include config.d/*).References and Relevant Issues
Fixes #19555
Detailed Description of the Pull Request / Additional comments
This PR implements SSH config
Includedirective support with the following features:Changes:
Includedirective parsing inSshHostGenerator::_getHostNamesFromConfigFile()_processIncludeDirective()to handle file inclusion with wildcard pattern support_matchesPattern()for wildcard matching (*and?patterns)Behavior:
Include ~/.ssh/config.d/workInclude config.d/*Implementation:
SshHostGenerator.cpp(+107 lines)SshHostGenerator.h(+2 function declarations)Validation Steps Performed
config.d/*)PR Checklist