Skip to content

fix: Added more verbose message for authentication failures#557

Merged
uipreliga merged 2 commits into
mainfrom
fix/auth-message
Sep 12, 2025
Merged

fix: Added more verbose message for authentication failures#557
uipreliga merged 2 commits into
mainfrom
fix/auth-message

Conversation

@uipreliga

@uipreliga uipreliga commented Sep 9, 2025

Copy link
Copy Markdown
Collaborator

This pull request updates the authentication error messages in the src/uipath/models/errors.py file to provide clearer guidance to users. The new messages inform users that they can either run uipath auth or set the appropriate environment variables to resolve authentication issues. Corresponding unit tests have been updated to reflect these changes.

Error message improvements:

  • Updated the default message in BaseUrlMissingError to mention providing a valid base URL via the UIPATH_URL environment variable.
  • Updated the default message in SecretMissingError to mention providing a valid secret via the UIPATH_ACCESS_TOKEN or UNATTENDED_USER_ACCESS_TOKEN environment variables.

Test updates:

  • Modified the test_no_config test assertion in tests/sdk/test_config.py to match the new error message for missing base URL.

@uipreliga uipreliga requested a review from cristipufu September 9, 2025 14:50
@uipreliga uipreliga changed the title Added more verbose message for authentication failures fix(errors): Added more verbose message for authentication failures Sep 9, 2025
@uipreliga uipreliga changed the title fix(errors): Added more verbose message for authentication failures fix: Added more verbose message for authentication failures Sep 9, 2025
@uipreliga uipreliga force-pushed the fix/auth-message branch 2 times, most recently from c777181 to 238afc7 Compare September 9, 2025 15:17
Comment thread src/uipath/models/errors.py Outdated

@cristipufu cristipufu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's also squash the commits please

Comment thread src/uipath/models/errors.py Outdated
@uipreliga uipreliga merged commit fd813a8 into main Sep 12, 2025
16 checks passed
@uipreliga uipreliga deleted the fix/auth-message branch September 12, 2025 23:14
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.

3 participants