Skip to content
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

[fix] Refactor to add auth error handling #177

Merged
merged 2 commits into from
Aug 22, 2024

Conversation

charliegerard
Copy link
Contributor

@charliegerard charliegerard commented Aug 21, 2024

This PR implements error handling on commands that require the user to be authenticated.
For example, if a user is not logged into the CLI and tries to run the command socket dependencies or socket organizations, the following error message will be displayed:

✖ Authentication error: User must be authenticated to run this command. To log in, run the command `socket login` and enter your API key.

@charliegerard charliegerard changed the title [wip] [fix] Refactor to add auth error handling [fix] Refactor to add auth error handling Aug 21, 2024
@charliegerard charliegerard marked this pull request as ready for review August 21, 2024 21:10
@charliegerard charliegerard marked this pull request as draft August 21, 2024 21:24
@charliegerard charliegerard changed the title [fix] Refactor to add auth error handling [wip] [fix] Refactor to add auth error handling Aug 21, 2024
@charliegerard charliegerard changed the title [wip] [fix] Refactor to add auth error handling [fix] Refactor to add auth error handling Aug 22, 2024
@charliegerard charliegerard marked this pull request as ready for review August 22, 2024 22:02
@charliegerard charliegerard merged commit ee2f2c4 into master Aug 22, 2024
11 checks passed
@charliegerard charliegerard deleted the cg/addAuthCheckDependencies branch August 22, 2024 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant