Skip to content

Add Swift-specific argument label for resumeExternalUserAgentFlow #966

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.

Sign up for GitHub

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

Merged
w-goog merged 4 commits into from
Jul 23, 2026
+7 −4
Merged

Add Swift-specific argument label for resumeExternalUserAgentFlow #966

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
ceb7e50
Add Swift-specific argument label for resumeExternalUserAgentFlow(_:e…
w-goog Jul 21, 2026
95d99ed
Add Swift-specific hint + update Changelog
w-goog Jul 21, 2026
c23e08d
Merge branch 'master' into fix-double-nullability-annotation
w-goog Jul 21, 2026
0ded869
Update Swift Carthage example as well.
w-goog Jul 23, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
# UNRELEASED
- Add Swift name to `resumeExternalUserAgentFlowWithURL:error:` and update hint. (#966)
- Replace case range with explicit case labels in OIDTokenUtilities. Addresses issue #947. ([#963](https://github.com/openid/AppAuth-iOS/pull/963))

# 2.1.0
Expand Down
2 changes: 1 addition & 1 deletion Examples/Example-iOS_Swift-SPM/Example/AppDelegate.swift
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ class AppDelegate: NSObject, UIApplicationDelegate {
// such as no pending flow, which previously surfaced as an NSException.
if let authorizationFlow = self.currentAuthorizationFlow {
do {
try authorizationFlow.resumeExternalUserAgentFlow(with: url)
try authorizationFlow.resumeExternalUserAgentFlow?(url)
Comment thread
w-goog marked this conversation as resolved.
self.currentAuthorizationFlow = nil
return true
} catch {
Expand Down
6 changes: 4 additions & 2 deletions Sources/AppAuthCore/OIDExternalUserAgentSession.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,9 @@ NS_ASSUME_NONNULL_BEGIN
@remarks Has no effect if called more than once, or after a @c cancel message was received.
@return YES if the passed URL matches the expected redirect URL and was consumed, NO otherwise.
*/
- (BOOL)resumeExternalUserAgentFlowWithURL:(NSURL *)URL __deprecated_msg("Use resumeExternalUserAgentFlowWithURL:error: instead");
- (BOOL)resumeExternalUserAgentFlowWithURL:(NSURL *)URL
__deprecated_msg("Use resumeExternalUserAgentFlowWithURL:error: instead. "
"Swift: Use the throwing resumeExternalUserAgentFlow(_:)");

@optional
/*! @brief Clients should call this method with the result of the external user-agent code flow if
Expand All @@ -71,7 +73,7 @@ NS_ASSUME_NONNULL_BEGIN
NO if the URL did not match (\@c OIDErrorCodeURLMismatch) or no authorization flow
was pending (\@c OIDErrorCodeInvalidAuthorizationFlow).
*/
- (BOOL)resumeExternalUserAgentFlowWithURL:(NSURL *)URL error:(NSError *_Nullable *_Nullable)error;
- (BOOL)resumeExternalUserAgentFlowWithURL:(NSURL *)URL error:(NSError *_Nullable *_Nullable)error NS_SWIFT_NAME(resumeExternalUserAgentFlow(_:));

@required
/*! @brief @c OIDExternalUserAgent or clients should call this method when the
Expand Down