-
Notifications
You must be signed in to change notification settings - Fork 19
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 CAA record #44
Add CAA record #44
Conversation
Signed-off-by: Pierre-Emmanuel Jacquier <pierre-emmanuel.jacquier@epitech.eu>
Signed-off-by: Pierre-Emmanuel Jacquier <pierre-emmanuel.jacquier@epitech.eu>
A small entry in the changelog would be greatly appreciated |
Signed-off-by: Pierre-Emmanuel Jacquier <pierre-emmanuel.jacquier@epitech.eu>
Co-Authored-By: pierre-emmanuelJ <pierre-emmanuel.jacquier@epitech.eu>
@@ -56,7 +56,7 @@ func init() { | |||
dnsAddCmd.AddCommand(dnsACmd) | |||
dnsACmd.Flags().StringP("name", "n", "", "Leave this blank to create a record for <domain name>, You may use the '*' wildcard here.") | |||
dnsACmd.Flags().StringP("address", "a", "", "Example: 127.0.0.1") | |||
dnsACmd.Flags().IntP("ttl", "t", 3600, "The time in second to leave (refresh rate) of the record.") | |||
dnsACmd.Flags().IntP("ttl", "t", 3600, "The time in seconds to live (refresh rate) of the record.") |
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.
😆
% exo dns add CAA yoanyoanyoan.io --tag issue,letsencrypt.org
Record "CAA" was created successfully to "yoanyoanyoan.io"
% exo dns show yoanyoanyoan.io CAA
|
copy(types, args[1:]) | ||
} else { | ||
types = append(types, "") | ||
types = []string{""} | ||
} |
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.
As the dst
was empty, nothing was copied.
Copy returns the number of elements copied, which will be the minimum of
len(src)
andlen(dst)
.
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.
thanks :)
Signed-off-by: Pierre-Emmanuel Jacquier <pierre-emmanuel.jacquier@epitech.eu>
Signed-off-by: Yoan Blanc <yoan.blanc@exoscale.ch>
(rebased) |
What is a flag? <https://tools.ietf.org/html/rfc6844#section-3> | ||
`, | ||
RunE: func(cmd *cobra.Command, args []string) error { | ||
if len(args) < 1 { |
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.
Maybe != 1
?
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.
It's not mandatory a the moment, because in other exo cmd it's like that for non fixed parameters and fixed parameters. So maybe another PR to refacto that. thank you
Signed-off-by: Pierre-Emmanuel Jacquier <pierre-emmanuel.jacquier@epitech.eu>
@@ -4,6 +4,7 @@ | |||
- feature new API call updateVmNicIp call | |||
- feature sos download has a progress bar | |||
- feature sos listings `--short` | |||
- feature dns CAA record |
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.
I'd rebase and put that into 1.0.4
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.
👍
Signed-off-by: Yoan Blanc <yoan.blanc@exoscale.ch>
Fix #42