-
Notifications
You must be signed in to change notification settings - Fork 89
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
Add request letsencrypt certificate #508
Add request letsencrypt certificate #508
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see my comment below on how Boolean flags should be passed
cli.js
Outdated
@@ -1047,6 +1047,8 @@ program | |||
.option('-h, --host <host>','hostname alias to register') | |||
.option('-j, --json', 'Optional, formats the output in json') | |||
.option('-u, --unique', 'Optional, define alias as unique, false by default') | |||
.option('-l, --request-letsencrypt-certificate <requestLetsncrypt>', 'Optional, ' + |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Boolean flags should be passed as name only arguments to overwrite the default value (false in this case). So, instead of:
sfcc-ci sandbox:alias:create --request-letsencrypt-certificate true
it should be
sfcc-ci sandbox:alias:create --request-letsencrypt-certificate
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
Thanks a lot @sanjaykesavan for the efforts! Could you please have a look at my comment (passing of Boolean flag)? |
Adds support for request of a Let's Encrypt certificate through the sandbox alias creation as new flag
-,--request-letsencrypt-certificate
to commandsfcc-ci sandbox:alias:create