feat: Add presentation window support by NandanPrabhu · Pull Request #1085 · auth0/Auth0.swift

NandanPrabhu · 2026-02-12T14:38:27Z

All new/changed/fixed functionality is covered by tests (or N/A)
I have added documentation for all new/changed functionality (or N/A)

📋 Changes

Added multi-window presentation support by plumbing optional UIWindow/NSWindow through ContentView, ContentViewModel, Auth0WebAuth, and all built-in web auth providers, plus enabling multi-scene mode in Info.plist.
Extended WebAuth API surface with presentationWindow(...), updated ASUserAgent, Safari/WKWebView providers, and ASWebAuthentication scaffolding to honor custom anchors on iOS, visionOS, and macOS.
Updated Auth0 defaults to request offline_access, set credentials manager minTTL to $60$, and give database signup a default "Username-Password-Authentication" connection, with aligned documentation updates and new Quick/Nimble specs covering default behaviors and presentation windows.

🎯 Testing

Ran unit suite including new WebAuthPresentationWindowSpec, extended AuthenticationSpec, and additional CredentialsManagerSpec cases to verify defaults and window handling.

Copilot

Pull request overview

This PR adds support for presenting Web Auth flows from a caller-specified window (UIWindow/NSWindow) to better support multi-window / multi-scene apps across iOS, visionOS, and macOS, and includes test coverage plus demo app wiring.

Changes:

Extended WebAuth/Auth0WebAuth + ASWebAuthenticationSession provider plumbing to accept an optional presentation window/anchor.
Updated Safari/WKWebView user agents to present from the top view controller of a provided UIWindow (fallback to key window).
Added Quick/Nimble specs validating presentationWindow(...) behavior and provider propagation; enabled multi-scene support in the demo app’s Info.plist.

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
Auth0Tests/WebAuthPresentationWindowSpec.swift	Adds specs for `presentationWindow(...)` and provider window propagation.
Auth0/WebAuth.swift	Extends `WebAuth` protocol with `presentationWindow(_:)` API (UIWindow/NSWindow).
Auth0/Auth0WebAuth.swift	Stores presentation window and passes it to the default AS provider for login/logout.
Auth0/ASProvider.swift	Extends AS provider factory + ASUserAgent to carry a presentation window and use it as anchor.
Auth0/MobileWebAuth.swift	Uses provided UIWindow as the `ASPresentationAnchor` on iOS/visionOS.
Auth0/DesktopWebAuth.swift	Uses provided NSWindow as the `ASPresentationAnchor` on macOS.
Auth0/SafariProvider.swift	Allows Safari user agent to present from a provided UIWindow.
Auth0/WebViewProvider.swift	Allows WKWebView user agent to present from a provided UIWindow.
Auth0/UIWindow+TopViewController.swift	Exposes a helper to find the top VC from a specific root VC.
App/Info.plist	Enables multiple scenes in the demo app.
App/ContentView.swift	Captures current UIWindow (iOS/visionOS) and passes it to login/logout calls.
App/ContentViewModel.swift	Adds window-aware login/logout variants and threads the window into WebAuth.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-12T16:35:04Z

Auth0Tests/WebAuthPresentationWindowSpec.swift

+#if os(iOS)
+import UIKit
+typealias PlatformWindow = UIWindow
+#elseif os(macOS)
+import AppKit
+typealias PlatformWindow = NSWindow
+#endif


UIWindow is referenced under #if os(iOS) || os(visionOS) but UIKit is only imported for os(iOS). This will fail to compile the test target on visionOS; update the conditional import to include visionOS (or avoid direct UIKit types in the visionOS blocks).

Copilot · 2026-02-12T16:35:04Z

App/ContentViewModel.swift

+#if os(iOS) || os(visionOS)
+import UIKit
+#endif
+


This file conditionally defines APIs using NSWindow for os(macOS), but AppKit is never imported. A macOS build of the demo app will fail to compile; add an #if os(macOS) import for AppKit (and keep UIKit under iOS/visionOS).

Suggested change

#if os(macOS)

import AppKit

#endif

Copilot · 2026-02-12T16:35:05Z

App/ContentViewModel.swift

+            // Use specific window for multi-window iPad support
+            #if os(iOS) || os(visionOS)
+            if let window = window {
+                webAuth = webAuth.presentationWindow(window)
+            }
+            #endif


In the os(macOS) implementation, the presentationWindow parameter is NSWindow? but the code that applies it is wrapped in #if os(iOS) || os(visionOS), so it never runs on macOS and the parameter becomes unused. Apply the window under the macOS branch (and update the comment about “UIWindow / iPad” accordingly).

Copilot · 2026-02-12T16:35:05Z

App/ContentViewModel.swift

+            // Use specific window for multi-window iPad support
+            #if os(iOS) || os(visionOS)
+            if let window = window {
+                webAuth = webAuth.presentationWindow(window)
+            }
+            #endif


Same issue as webLogin: in the os(macOS) logout implementation, the NSWindow? parameter is only used inside an #if os(iOS) || os(visionOS) block, so it will never be applied on macOS. Use the macOS presentationWindow(_:) API here (and update the doc comment to refer to NSWindow).

Copilot · 2026-02-12T16:35:05Z

App/ContentView.swift

+    #if os(macOS)
+    @State private var currentWindow: NSWindow?
+    #else
+    @State private var currentWindow: UIWindow?
+    #endif


NSWindow is referenced under #if os(macOS) but AppKit is not imported anywhere in this file. If the demo app is built for macOS, this will not compile; add an #if os(macOS) import for AppKit.

Copilot · 2026-02-12T16:35:05Z

App/ContentView.swift

+            Button {
+                Task {
+                    #if !os(macOS)
+                    await viewModel.webLogin(presentationWindow: currentWindow)
+                    #endif
+                }


Both button actions are compiled out on macOS (#if !os(macOS)), so the demo app UI does nothing on macOS even though ContentViewModel provides macOS-specific webLogin/logout overloads. Either call the macOS overloads under #if os(macOS) or remove the macOS branches from the view/model to avoid dead code paths.

Copilot · 2026-02-12T16:35:06Z

App/ContentView.swift

+        .onAppear {
+            // Capture the current window for multi-window iPad support
+#if os(iOS) || os(visionOS)
+            currentWindow = getCurrentWindow()
+#endif
+        }


On macOS, currentWindow is never populated (the .onAppear window capture is iOS/visionOS-only). If macOS support is intended, populate currentWindow from NSApp.keyWindow/NSApplication.shared.keyWindow (or remove currentWindow on macOS if it’s not needed).

sanchitmehtagit · 2026-02-13T08:54:09Z

I have added mandate PR CI checks and 1 mandatory approval for develop/v3 branch too.
cc: @NandanPrabhu

sanchitmehtagit · 2026-02-13T09:07:10Z

App/ContentView.swift

 import Auth0

+#if os(iOS) || os(visionOS)
+import UIKit


why do we have optional UIKit import here?

Do we need to import AppKit for NSWindow in MacOS?

sanchitmehtagit · 2026-02-13T09:30:19Z

Auth0/Auth0WebAuth.swift

    private(set) var provider: WebAuthProvider?
    private(set) var onCloseCallback: (() -> Void)?
+    #if os(macOS)
+    private(set) var presentationWindow: NSWindow?


We should store the weak references for these since the window is owned by the application, not by the Auth0WebAuth instance.
private(set) weak var presentationWindow: NSWindow?
private(set) var presentationWindow: UIWindow?

sanchitmehtagit · 2026-02-13T09:35:46Z

Auth0/WebViewProvider.swift

    let callback: WebAuthProviderCallback
-
-    init(authorizeURL: URL, redirectURL: URL, viewController: UIViewController = UIViewController(), modalPresentationStyle: UIModalPresentationStyle = .fullScreen, callback: @escaping WebAuthProviderCallback) {
+    var presentationWindow: UIWindow?


we should be using weak reference in SDK

this should be private(set)

sanchitmehtagit · 2026-02-13T09:49:50Z

Auth0/UIWindow+TopViewController.swift

    }

-    private static func findTopViewController(from root: UIViewController) -> UIViewController? {
+    static func findTopViewController(from root: UIViewController) -> UIViewController? {


should we make this private back .

sanchitmehtagit · 2026-02-13T10:09:36Z

App/ContentViewModel.swift

+
    /// Logout and clear stored credentials
-    func logout() async {
+    /// - Parameter window: The UIWindow to present the authentication browser. Used for multi-window iPad apps.


Do update this comment . This API is being used for MacOS

sanchitmehtagit · 2026-02-13T10:58:06Z

App/ContentViewModel.swift

+    #if os(macOS)
    /// Web Authentication using Universal Login (Recommended)
-    func webLogin() async {
+    /// - Parameter window: The UIWindow to present the authentication browser. Used for multi-window iPad apps.


this paramter is being used for MacOS

sanchitmehtagit · 2026-02-13T11:07:21Z

App/ContentViewModel.swift

+            var webAuth = Auth0.webAuth()
+
+            // Use specific window for multi-window iPad support
+            #if os(iOS) || os(visionOS)


#if os(iOS) || os(visionOS) / if let window = window { webAuth = webAuth.presentationWindow(window) } #endif

This will never execute as the current logout function will be executed for MacOS

sanchitmehtagit · 2026-02-13T11:08:51Z

App/ContentViewModel.swift

-
+
+            // Use specific window for multi-window iPad support
+            #if os(iOS) || os(visionOS)


#if os(iOS) || os(visionOS) / if let window = window { webAuth = webAuth.presentationWindow(window) } #endif

This will never execute as the current login function will be executed for MacOS

sanchitmehtagit · 2026-02-13T11:11:52Z

App/ContentViewModel.swift

+    /// - Parameter window: The UIWindow to present the authentication browser. Used for multi-window iPad apps.
+    func webLogin(presentationWindow window: NSWindow? = nil) async {
        isLoading = true
        errorMessage = nil


Lets update the documentation . We should have Multi Window Support in examples.md file

sanchitmehtagit · 2026-02-13T11:18:50Z

Auth0/ASProvider.swift

+    #endif
+
+    #if os(macOS)
+    init(session: ASWebAuthenticationSession,


we should not add have 2 initialiser implementation

Please user the the below design. It improved readability and eliminates the code duplicity

#if os(macOS) typealias PlatformWindow = NSWindow #else typealias PlatformWindow = UIWindow #endif class ASUserAgent: NSObject, WebAuthUserAgent { var presentationWindow: PlatformWindow? init(session: ASWebAuthenticationSession, callback: @escaping WebAuthProviderCallback, presentationWindow: PlatformWindow? = nil) { // ... single implementation } }

sanchitmehtagit · 2026-02-13T11:23:53Z

Auth0/Auth0WebAuth.swift

    }

+    #if os(macOS)
+    @available(macOS 10.15, *)


#if os(macOS) typealias PlatformWindow = NSWindow #else typealias PlatformWindow = UIWindow #endif final class Auth0WebAuth: WebAuth { // ... properties ... private(set) weak var presentationWindow: PlatformWindow? // ... later in the file ... @available(iOS 13.0, macOS 10.15, visionOS 1.0, *) func presentationWindow(_ window: PlatformWindow) -> Self { self.presentationWindow = window return self } }

Preferred design we should use same approach everywhere

Eliminates duplicate property declarations

Single method definition instead of two platform-specific versions

Easier to maintain - changes only need to be made once

…troller WKWebView

sanchitmehtagit · 2026-02-17T05:18:46Z

Auth0/MobileWebAuth.swift

    func presentationAnchor(for session: ASWebAuthenticationSession) -> ASPresentationAnchor {
-        return UIApplication.shared()?.windows.last(where: \.isKeyWindow) ?? ASPresentationAnchor()
+        // Use custom window if provided
+        if let window = presentationWindow {


could we use guard here

sanchitmehtagit · 2026-02-17T05:19:15Z

Auth0/MobileWebAuth.swift

 #if os(visionOS)
    func presentationAnchor(for session: ASWebAuthenticationSession) -> ASPresentationAnchor {
+        // Use custom window if provided
+        if let window = presentationWindow {


Take window from client to handle multi window case for iPad

dd45303

NandanPrabhu requested a review from a team as a code owner February 12, 2026 14:38

macos fixes

daa5c6d

pmathew92 requested a review from Copilot February 12, 2026 16:25

Copilot started reviewing on behalf of pmathew92 February 12, 2026 16:26 View session

Copilot AI reviewed Feb 12, 2026

View reviewed changes

sanchitmehtagit reviewed Feb 13, 2026

View reviewed changes

sanchitmehtagit added the review:medium Medium review label Feb 13, 2026

sanchitmehtagit changed the title ~~Add presentation window support~~ feat: Add presentation window support Feb 13, 2026

sanchitmehtagit reviewed Feb 13, 2026

View reviewed changes

NandanPrabhu added 2 commits February 15, 2026 20:45

Resolves comments Handles windows hanlding for macOS, SFSafariViewCon…

a272233

…troller WKWebView

Updates .net mobie iOS and android

6dbd6fd

sanchitmehtagit reviewed Feb 17, 2026

View reviewed changes



		// Use specific window for multi-window iPad support
		#if os(iOS) \|\| os(visionOS)

Conversation

NandanPrabhu commented Feb 12, 2026

📋 Changes

🎯 Testing

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

sanchitmehtagit commented Feb 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sanchitmehtagit Feb 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

sanchitmehtagit commented Feb 13, 2026 •

edited

Loading

sanchitmehtagit Feb 13, 2026 •

edited

Loading