Release v4.0.0 #1329
Merged
Release v4.0.0 #1329
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Remove the "default requested scopes" These were the default for login flows and client credentials. When GlobusApp-driven logins are used, these defaults are not used. They were only applied for direct use of the login flow and token request mechanisms for login clients. One log line has been changed to produce slightly more useful debug-level logs. Also, introduce the first v3.x -> v4 upgrading section to the upgrading doc. * Fix typing errors and raise on unreachable types Fix typing errors where `... | None` was passed for `requested_scopes`. Additionally, to handle cases in GlobusApp, raise usage errors if an app is ever doing a login without provided scopes in the authorization parameters. These conditions aren't unreachable based on the types which are used, but a user who configures things to reach these conditions is engaged in explicitly unsupported usage (e.g., trying to give explicit parameters to a client app which don't include scopes). * Retitle changelog section
* Removed base_path from client classes * Removed _base_path_map from RegisteredResponse * Removed trailing slash enforcement * Added changelog
In the source tree, the default is changed from `"Error"` to `None`. In the tests, various direct tests are updated, as are those which inspect the arguments passed to `super().__init__` and which feed into the repr for an API error. There are no paths in the source tree of the SDK itself which depend upon the old default of `"Error"`.
…ectionDocument Updated CollectionDocument with MissingType
In particular, remove the special-case `code` default. Also, make use of subdocument message parsing to capture the `msg` field of sub-errors.
Release v4.0.0a1
Remove the `globus_sdk.version` module
* Remove the 'MutableScope' type This is deprecated in 3.x in favor of the 'Scope' type. As part of our 4.0 release, we will be removing the 'MutableScope' type. This removes the type itself, and then handles all of the cascading fixes this necessitates. No update to the upgrading doc is included yet, as we have additional plans to make changes to the scope interfaces. * Minor fix to a mypy-test `Scope.scopes2str` does not exist -- this test was created by a `MutableScope` replacement but the `Scope` and `MutableScope` interfaces differ here. Remove the bad test case.
This is a small/minimal entry so that we're guaranteed to have something to build upon. It intentionally does not go into detail on how to use `Scope` because these interfaces are being redesigned.
Fix some phrasing and link to classes. Co-authored-by: Kurt McKee <39996+kurtmckee@users.noreply.github.com>
…upgrade Add an entry to the upgrading doc for MutableScope
- Remove `globus_sdk.experimental.auth_requirements_error` - Remove a test which checks the aliasing - Add an upgrading doc section on removal of experimental aliases
Co-authored-by: Kurt McKee <39996+kurtmckee@users.noreply.github.com>
Remove legacy "GARE" module aliasing
This was deprecated under 3.x and is now being removed. It is sufficiently minor that it probably doesn't warrant a call-out in the upgrading guide doc.
* Updated flows client to use MISSING defaults * Added changelog * Requested changes: removed missing from additional_fields and query_params * Requested changes: added back orderby comment
Several objects named `Timer*` were exposed in the SDK as deprecated aliases for their `Timers*` renamed variants. These aliases are here removed. Also, delete the `globus_sdk.services.timer` module which aliased them. Tests and docs are updated to remove all reference to these old names. Co-authored-by: Kurt McKee <39996+kurtmckee@users.noreply.github.com>
v3.57.0 Merge resolution against the GCS collection types required rewrites of the new fields into `MISSING`-style.
'activity_notification_policy' was being set twice. This was an accidental merge artifact, fixed here.
Merge version 3.57.0 into 4.x-dev
This takes the latest 3.x development work (`main`) and merges it into 4.x development (`4.x-dev`). In particular, this changeset takes globus#1307 and globus#1308 and applies them against 4.x. The following changes are made in order to resolve merge conflicts and produce a good result: - the deprecated method and parameter from globus#1307 are removed - the deprecated parameters from globus#1308 are removed - tests for deprecated behaviors are removed - the relevant changelogs are removed from the `changelog.d/` directory - a new changelog.d/ fragment is added to describe the removals
…Data and DeleteData (merge 3.x main to 4.x-dev) (globus#1309)
Because this only impacts users who are directly using HTTP methods, the expected impact of this change is somewhat limited. As such, it is placed at the end of the guide. In brief, the guide lists which clients and which methods of said clients are impacted, as well as giving notes on: - how the mapping of usages to URIs has changed - impact to the testing tools - compatible usage which does the same thing on both versions
sirosen
requested review from
MaxTueckeGlobus,
aaschaer,
ada-globus,
derek-globus,
kurtmckee and
m1yag1
as code owners
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Commits in 4.x which are not in 3.x are listed below.
globus_sdk.versionmoduleadd_app_transfer_data_access_scopefor flowsutilsprovisionflag clearertokenstoragetotoken_storage(Renametokenstoragetotoken_storage#1252)globus_sdk._testingtoglobus_sdk.testing(Move globus_sdk._testing to globus_sdk.testing #1251)create_policy(Remove the compatibility shim fromcreate_policy#1257)ScopeCollectionTypealias with explicit expansion of types (ReplaceScopeCollectionTypealias with explicit expansion of types #1259)UUIDLiketype aliasConfidentialAppAuthClient.get_identitiesGCSClient.connector_id_to_nameBaseClient.transportis customizedRetryConfigurationwarn_deprecatedto emitRemovedInV5Warning__notes__where supportedtox p