Skip to content

Fix incorrect message during auth#275

Merged
jai-deepsource merged 2 commits intomasterfrom
login-fix
Mar 5, 2026
Merged

Fix incorrect message during auth#275
jai-deepsource merged 2 commits intomasterfrom
login-fix

Conversation

@jai-deepsource
Copy link
Contributor

No description provided.

- Skip the re-authenticate prompt when no token exists in config
- After logout, config file is deleted so token is empty string
- Previously IsExpired() returned false for zero time, causing the
  prompt to show even with no token
@jai-deepsource jai-deepsource self-assigned this Mar 5, 2026
@deepsource-io
Copy link

deepsource-io bot commented Mar 5, 2026

DeepSource Code Review

We reviewed changes in fccfbea...af4b0d1 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Coverage  

Code Review Summary

Analyzer Status Updated (UTC) Details
Go Mar 5, 2026 6:17p.m. Review ↗
Secrets Mar 5, 2026 6:17p.m. Review ↗
Test coverage Mar 5, 2026 6:17p.m. Review ↗

Code Coverage Summary

Language Line Coverage (New Code) Line Coverage (Overall)
Aggregate
0%
[⤫ below threshold]
19.2%
Go
0%
[⤫ below threshold]
19.2%
[✓ above threshold]

➟ Additional coverage metrics may have been reported. See full coverage report ↗

@jai-deepsource jai-deepsource merged commit 383bb7a into master Mar 5, 2026
4 of 5 checks passed
@jai-deepsource jai-deepsource deleted the login-fix branch March 5, 2026 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant