Merge pull request #137 from docker/fix-sudo-commands

Do not try to 2FA if the user is not logged-in
This commit is contained in:
Silvin Lubecki 2020-11-20 18:16:15 +01:00 коммит произвёл GitHub
Родитель 8ad1a0dc0d 2d6647521a
Коммит 06a23b6d7b
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: 4AEE18F83AFDEB23
1 изменённых файлов: 4 добавлений и 4 удалений

Просмотреть файл

@ -70,15 +70,15 @@ func NewRootCmd(streams command.Streams, hubClient *hub.Client, store credential
return nil
}
if cmd.Annotations["sudo"] == "true" {
return requireTwoFactorCode(cmd.Context(), streams, hubClient, store)
}
ac, err := store.GetAuth()
if err != nil {
return err
}
if cmd.Annotations["sudo"] == "true" && ac.Username != "" {
return requireTwoFactorCode(cmd.Context(), streams, hubClient, store)
}
if ac.Username == "" {
log.Fatal(ansi.Error(`You need to be logged in to Docker Hub to use this tool.
Please login to Docker Hub using the "hub-tool login" command.`))