Skip to content

Commit e80779d

Browse files
authored
Merge pull request #18384 from mozilla/FXA-11151
fix(auth): SMS message did not include body text
2 parents f641ada + 80f0592 commit e80779d

1 file changed

Lines changed: 7 additions & 7 deletions

File tree

packages/fxa-auth-server/lib/routes/recovery-phone.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ class RecoveryPhoneHandler {
6060
this.localizer = new Localizer(new NodeRendererBindings());
6161
}
6262

63-
getLocalizedStrings = async (
63+
getLocalizedMessage = async (
6464
request: AuthRequest,
6565
code: string,
6666
type: 'setup' | 'signin'
@@ -85,7 +85,7 @@ class RecoveryPhoneHandler {
8585
},
8686
]
8787
);
88-
return localizedStrings;
88+
return localizedStrings[id];
8989
};
9090

9191
async sendCode(request: AuthRequest) {
@@ -102,11 +102,11 @@ class RecoveryPhoneHandler {
102102
await this.customs.check(request, email, 'recoveryPhoneSendCode');
103103

104104
const getFormattedMessage = async (code: string) => {
105-
const localizedMessage = await this.getLocalizedStrings(
105+
const localizedMessage = await this.getLocalizedMessage(
106106
request,
107107
code,
108108
'signin'
109-
)[0];
109+
);
110110
return localizedMessage;
111111
};
112112

@@ -169,11 +169,11 @@ class RecoveryPhoneHandler {
169169
await this.customs.checkAuthenticated(request, uid, 'recoveryPhoneCreate');
170170

171171
const getFormattedMessage = async (code: string) => {
172-
const localizedMessage = await this.getLocalizedStrings(
172+
const localizedMessage = await this.getLocalizedMessage(
173173
request,
174174
code,
175175
'setup'
176-
)[0];
176+
);
177177
return localizedMessage;
178178
};
179179

@@ -498,7 +498,7 @@ class RecoveryPhoneHandler {
498498
} catch (error) {
499499
throw AppError.backendServiceFailure(
500500
'RecoveryPhoneService',
501-
'destroy',
501+
'available',
502502
{ uid },
503503
error
504504
);

0 commit comments

Comments
 (0)