Skip to content

Commit 49b0406

Browse files
authored
[Auth] Fix incorrectly-cased parameter in OOB request (#5542)
* Fix incorrect casing on iOSBundleId in action_code_settings * Fix tests * Add changeset
1 parent 316638a commit 49b0406

File tree

6 files changed

+11
-6
lines changed

6 files changed

+11
-6
lines changed

.changeset/swift-items-prove.md

+5
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@firebase/auth": patch
3+
---
4+
5+
Fix incorrectly-cased parameter in out-of-band request that was causing incorrect behavior in some cases

packages/auth/src/api/authentication/email_and_password.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ export async function signInWithPassword(
5656
export interface GetOobCodeRequest {
5757
email?: string; // Everything except VERIFY_AND_CHANGE_EMAIL
5858
continueUrl?: string;
59-
iosBundleId?: string;
59+
iOSBundleId?: string;
6060
iosAppStoreId?: string;
6161
androidPackageName?: string;
6262
androidInstallApp?: boolean;

packages/auth/src/core/strategies/action_code_settings.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ export function _setActionCodeSettingsOnRequest(
4848
auth,
4949
AuthErrorCode.MISSING_IOS_BUNDLE_ID
5050
);
51-
request.iosBundleId = actionCodeSettings.iOS.bundleId;
51+
request.iOSBundleId = actionCodeSettings.iOS.bundleId;
5252
}
5353

5454
if (actionCodeSettings.android) {

packages/auth/src/core/strategies/email.test.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,7 @@ describe('core/strategies/sendEmailVerification', () => {
171171
continueUrl: 'my-url',
172172
dynamicLinkDomain: 'fdl-domain',
173173
canHandleCodeInApp: true,
174-
iosBundleId: 'my-bundle'
174+
iOSBundleId: 'my-bundle'
175175
});
176176
});
177177
});
@@ -280,7 +280,7 @@ describe('core/strategies/verifyBeforeUpdateEmail', () => {
280280
continueUrl: 'my-url',
281281
dynamicLinkDomain: 'fdl-domain',
282282
canHandleCodeInApp: true,
283-
iosBundleId: 'my-bundle'
283+
iOSBundleId: 'my-bundle'
284284
});
285285
});
286286
});

packages/auth/src/core/strategies/email_and_password.test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ describe('core/strategies/sendPasswordResetEmail', () => {
104104
continueUrl: 'my-url',
105105
dynamicLinkDomain: 'fdl-domain',
106106
canHandleCodeInApp: true,
107-
iosBundleId: 'my-bundle'
107+
iOSBundleId: 'my-bundle'
108108
});
109109
});
110110
});

packages/auth/src/core/strategies/email_link.test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ describe('core/strategies/sendSignInLinkToEmail', () => {
119119
continueUrl: 'my-url',
120120
dynamicLinkDomain: 'fdl-domain',
121121
canHandleCodeInApp: true,
122-
iosBundleId: 'my-bundle'
122+
iOSBundleId: 'my-bundle'
123123
});
124124
});
125125
});

0 commit comments

Comments
 (0)