fix(auth): missing profile client references

Because:

- Some code was still referencing the auth-server profile client

This commit:

- Update code to reference the profile client library

Closes #FXA-10389
This commit is contained in:
Reino Muhl 2024-09-05 16:02:42 -04:00
Родитель 788bf5b514
Коммит 7bed17ba8e
Не найден ключ, соответствующий данной подписи
2 изменённых файлов: 4 добавлений и 10 удалений

Просмотреть файл

@ -33,9 +33,8 @@ import {
appStoreSubscriptionPurchaseToAppStoreSubscriptionDTO, appStoreSubscriptionPurchaseToAppStoreSubscriptionDTO,
playStoreSubscriptionPurchaseToPlayStoreSubscriptionDTO, playStoreSubscriptionPurchaseToPlayStoreSubscriptionDTO,
} from '../payments/iap/iap-formatter'; } from '../payments/iap/iap-formatter';
import { PayPalHelper } from '../payments/paypal/helper';
import { StripeHelper } from '../payments/stripe'; import { StripeHelper } from '../payments/stripe';
import { AuthLogger, AuthRequest, ProfileClient } from '../types'; import { AuthLogger, AuthRequest } from '../types';
import { deleteAccountIfUnverified } from './utils/account'; import { deleteAccountIfUnverified } from './utils/account';
import emailUtils from './utils/email'; import emailUtils from './utils/email';
import requestHelper from './utils/request_helper'; import requestHelper from './utils/request_helper';
@ -45,6 +44,7 @@ import { gleanMetrics } from '../metrics/glean';
import { AccountDeleteManager } from '../account-delete'; import { AccountDeleteManager } from '../account-delete';
import { uuidTransformer } from 'fxa-shared/db/transformers'; import { uuidTransformer } from 'fxa-shared/db/transformers';
import { AccountTasks, ReasonForDeletion } from '@fxa/shared/cloud-tasks'; import { AccountTasks, ReasonForDeletion } from '@fxa/shared/cloud-tasks';
import { ProfileClient } from '@fxa/profile/client';
const METRICS_CONTEXT_SCHEMA = require('../metrics/context').schema; const METRICS_CONTEXT_SCHEMA = require('../metrics/context').schema;
@ -58,7 +58,6 @@ const MS_ONE_MONTH = MS_ONE_DAY * 30;
export class AccountHandler { export class AccountHandler {
private OAUTH_DISABLE_NEW_CONNECTIONS_FOR_CLIENTS: Set<string>; private OAUTH_DISABLE_NEW_CONNECTIONS_FOR_CLIENTS: Set<string>;
private paypalHelper?: PayPalHelper;
private otpUtils: any; private otpUtils: any;
private otpOptions: ConfigType['otp']; private otpOptions: ConfigType['otp'];
private skipConfirmationForEmailAddresses: string[]; private skipConfirmationForEmailAddresses: string[];
@ -94,12 +93,6 @@ export class AccountHandler {
this.otpOptions = config.otp; this.otpOptions = config.otp;
if (
stripeHelper &&
config.subscriptions?.paypalNvpSigCredentials?.enabled
) {
this.paypalHelper = Container.get(PayPalHelper);
}
this.capabilityService = Container.get(CapabilityService); this.capabilityService = Container.get(CapabilityService);
this.accountEventsManager = Container.get(AccountEventsManager); this.accountEventsManager = Container.get(AccountEventsManager);
this.accountDeleteManager = Container.get(AccountDeleteManager); this.accountDeleteManager = Container.get(AccountDeleteManager);

Просмотреть файл

@ -32,8 +32,9 @@ const {
const { const {
deleteAccountIfUnverified, deleteAccountIfUnverified,
} = require('../../../lib/routes/utils/account'); } = require('../../../lib/routes/utils/account');
const { AppConfig, AuthLogger, ProfileClient } = require('../../../lib/types'); const { AppConfig, AuthLogger } = require('../../../lib/types');
const defaultConfig = require('../../../config').default.getProperties(); const defaultConfig = require('../../../config').default.getProperties();
const { ProfileClient } = require('@fxa/profile/client');
const glean = mocks.mockGlean(); const glean = mocks.mockGlean();
const profile = mocks.mockProfile(); const profile = mocks.mockProfile();