Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

fix: Added more verbose message for authentication failures#557

Merged
uipreliga merged 2 commits intomainUiPath/uipath-python:mainfrom
fix/auth-messageUiPath/uipath-python:fix/auth-messageCopy head branch name to clipboard
Sep 12, 2025
Merged

fix: Added more verbose message for authentication failures#557
uipreliga merged 2 commits intomainUiPath/uipath-python:mainfrom
fix/auth-messageUiPath/uipath-python:fix/auth-messageCopy head branch name to clipboard

Conversation

@uipreliga
Copy link
Collaborator

@uipreliga uipreliga commented Sep 9, 2025

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
src/uipath/models/errors.py Outdated Show resolved Hide resolved
Copy link
Member

@cristipufu cristipufu left a comment

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

src/uipath/models/errors.py Outdated Show resolved Hide resolved
@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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.