Skip to content
This repository was archived by the owner on Feb 10, 2023. It is now read-only.

Commit aef73a9

Browse files
clean up login
1 parent ddef349 commit aef73a9

File tree

2 files changed

+12
-16
lines changed

2 files changed

+12
-16
lines changed

internal/secret/store.go

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -11,31 +11,27 @@ type SecretStore struct {
1111
ring keyring.Keyring
1212
}
1313

14-
func (s *SecretStore) LoadCredentials() (string, string, string, error) {
14+
func (s *SecretStore) LoadCredentials() (string, string, error) {
1515
if s.ring == nil {
1616
err := s.openRing()
1717
if err != nil {
18-
return "", "", "", err
18+
return "", "", err
1919
}
2020
}
2121

2222
if s.ring == nil {
23-
return "", "", "", errors.New("No keyring configured")
23+
return "", "", errors.New("No keyring configured")
2424
}
2525
username, err := s.ring.Get("username")
2626
if err != nil {
27-
return "", "", "", fmt.Errorf("Could not get username: %w", err)
27+
return "", "", fmt.Errorf("Could not get username: %w", err)
2828
}
2929

3030
password, err := s.ring.Get("password")
3131
if err != nil {
32-
return "", "", "", fmt.Errorf("Could not get password: %w", err)
32+
return "", "", fmt.Errorf("Could not get password: %w", err)
3333
}
34-
token, err := s.ring.Get("skylarkToken")
35-
if err != nil {
36-
return "", "", "", fmt.Errorf("Could not get auth token: %w", err)
37-
}
38-
return string(username.Data), string(password.Data), string(token.Data), nil
34+
return string(username.Data), string(password.Data), nil
3935
}
4036

4137
func (s *SecretStore) SaveCredentials(username, password, authtoken string) error {

internal/ui/state.go

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -177,26 +177,26 @@ func (s *UIState) logout() {
177177
}
178178

179179
func (s *UIState) loginWithStoredCredentials() error {
180-
username, password, token, err := s.secret.LoadCredentials()
180+
username, password, err := s.secret.LoadCredentials()
181181
if err != nil {
182182
return err
183183
}
184-
return s.login(username, password, token)
184+
return s.login(username, password)
185185
}
186186

187-
func (s *UIState) login(username, pw, token string) error {
187+
func (s *UIState) login(username, pw string) error {
188188
// todo save auth token in credential store
189189
err := s.v2.Authenticate(username, pw, s.logger)
190190
return err
191191
}
192192

193193
func (s *UIState) initUIWithForm() {
194-
username, pw, _, _ := s.secret.LoadCredentials()
194+
username, pw, _ := s.secret.LoadCredentials()
195195
form := tview.NewForm().
196196
AddInputField("email", username, 30, nil, func(text string) { username = text }).
197197
AddPasswordField("password", "", 30, '*', func(text string) { pw = text }).
198198
AddButton("test", func() {
199-
err := s.login(username, pw, "")
199+
err := s.login(username, pw)
200200
if err == nil {
201201
s.logger.Info("credentials accepted")
202202
} else {
@@ -245,7 +245,7 @@ func (s *UIState) initUI() {
245245
}
246246

247247
func (s *UIState) closeForm(username, pw string) {
248-
err := s.login(username, pw, "")
248+
err := s.login(username, pw)
249249
if err != nil {
250250
s.logger.Error(err)
251251
} else {

0 commit comments

Comments
 (0)