Skip to content

Commit 7d3a504

Browse files
authored
fix: string.Empty causes error in BankId (#519)
Update default returnUrl from String.Empty to null to comply with new BankID requirements and prevent 400 errors
1 parent 8a49118 commit 7d3a504

1 file changed

Lines changed: 3 additions & 5 deletions

File tree

src/ActiveLogin.Authentication.BankId.Core/Launcher/BankIdLauncher.cs

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -170,12 +170,10 @@ private static string GetIOsBrowserSpecificRedirectUrl(BankIdSupportedDevice dev
170170
BankIdSupportedDeviceBrowser.Chrome => IosChromeScheme,
171171
BankIdSupportedDeviceBrowser.Firefox => IosFirefoxScheme,
172172

173-
// Opens a new tab on app launch, so can't launch automatically
174-
BankIdSupportedDeviceBrowser.Edge => string.Empty,
175-
BankIdSupportedDeviceBrowser.Opera => string.Empty,
173+
BankIdSupportedDeviceBrowser.Edge => NullRedirectUrl,
174+
BankIdSupportedDeviceBrowser.Opera => NullRedirectUrl,
176175

177-
// Return empty string so user can go back manually, will catch unknown third party browsers
178-
_ => string.Empty
176+
_ => NullRedirectUrl
179177
};
180178
}
181179

0 commit comments

Comments
 (0)