-
Notifications
You must be signed in to change notification settings - Fork 29
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
Release 0.2.3 #72
Release 0.2.3 #72
Conversation
Merge MSAL Extensions for Python 0.2.2 back to dev branch
Add CodeQL Analysis workflow
Move .github/workflows/workflows dir to .github/workflows
Handle file not found exceptions on Python 2.7
* Unified exception when load() is called before save() * Go with an explicit PersistenceNotFound exception * Caller has a much simpler pattern to handle read errors * Change "where" to "location" * Test cases for PersistentNotFound
Change PersistenceNotFound's base from OSError to IOError
Bumping version number
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's close this one and start over as 0.3.0
if data is None: | ||
# Lower level libsecret would return None when found nothing. Here | ||
# in persistence layer, we convert it to a unified error for consistence. | ||
raise PersistenceNotFound(message=( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My bad. Although all other new PersistenceNotFound are not considered as breaking change, this particular one is. https://github.com/AzureAD/microsoft-authentication-extensions-for-python/pull/67/files#diff-3ef6fda259d01e5bd78b0a594e5f9f01R263-R264
Please abandon the 0.2.3 and let's go with 0.3.0
Closing this PR as we decided to do a 0.3.0 release instead |
PersistenceNotFound
exception is now raised for cases where the persistence is not found. (Investigate OS errors returned for encryption scenarios on Windows and MacOS #64, Unified exception when load() is called before save() #67)