fix(web): обратный поток из контура — vitest 3.1.4, форма dev-входа в любой сборке (#92)
- vitest зафиксирован точно на 3.1.4: в зеркале npm банка нет 3.2.x, lock-файл не в git, каретка давала бы разные версии снаружи и внутри контура; - DevLogin больше не выбирает форму по NODE_ENV: платформа всегда собирает фронт в production, а контур работает на dev-заглушке аутентификации, переменные сборки в пайплайне задать нельзя. Форма входа по email показывается всегда; заглушка «Вход выполняется через РАИСА» — только после 404 «Not Found» от бэкенда (маршрута /dev/token нет → dev-эмитент выключен). Признак вынесен в isDevIssuerDisabled и покрыт тестами: 404 «пользователь не найден» остаётся ошибкой в форме. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This commit is contained in:
parent
71e55b19ca
commit
d065fc31dd
@ -30,7 +30,7 @@
|
|||||||
"daisyui": "^4.12.10",
|
"daisyui": "^4.12.10",
|
||||||
"tailwindcss": "^3.4.13",
|
"tailwindcss": "^3.4.13",
|
||||||
"typescript": "^5.7.3",
|
"typescript": "^5.7.3",
|
||||||
"vitest": "^3.2.7"
|
"vitest": "3.1.4"
|
||||||
},
|
},
|
||||||
"optionalDependencies": {
|
"optionalDependencies": {
|
||||||
"@biomejs/cli-linux-x64-musl": "^1.9.4",
|
"@biomejs/cli-linux-x64-musl": "^1.9.4",
|
||||||
|
|||||||
@ -2,12 +2,14 @@ import { Alert, Button, Form, Input } from 'antd';
|
|||||||
import { useState } from 'react';
|
import { useState } from 'react';
|
||||||
import { api } from '../api/client';
|
import { api } from '../api/client';
|
||||||
import type { DevTokenResponse } from '../api/types';
|
import type { DevTokenResponse } from '../api/types';
|
||||||
|
import { isDevIssuerDisabled } from './devIssuer';
|
||||||
import { setToken } from './token';
|
import { setToken } from './token';
|
||||||
|
|
||||||
type Props = { onLogin: () => void; error?: string | null };
|
type Props = { onLogin: () => void; error?: string | null };
|
||||||
|
|
||||||
// Вход по email для разработки: бэкенд выпускает токен только при AUTH_DEV_ISSUER=true и DEBUG=true.
|
// Вход по email для разработки и контура без SSO: бэкенд выпускает токен только при AUTH_DEV_ISSUER=true и DEBUG=true.
|
||||||
function DevLoginForm({ onLogin, error }: Props) {
|
// `onDisabled` — бэкенд ответил, что маршрута /dev/token нет (dev-эмитент выключен): вход возможен только через SSO.
|
||||||
|
function DevLoginForm({ onLogin, error, onDisabled }: Props & { onDisabled: () => void }) {
|
||||||
const [submitting, setSubmitting] = useState(false);
|
const [submitting, setSubmitting] = useState(false);
|
||||||
const [failure, setFailure] = useState<string | null>(null);
|
const [failure, setFailure] = useState<string | null>(null);
|
||||||
|
|
||||||
@ -19,6 +21,10 @@ function DevLoginForm({ onLogin, error }: Props) {
|
|||||||
setToken(access_token);
|
setToken(access_token);
|
||||||
onLogin();
|
onLogin();
|
||||||
} catch (e: unknown) {
|
} catch (e: unknown) {
|
||||||
|
if (isDevIssuerDisabled(e)) {
|
||||||
|
onDisabled();
|
||||||
|
return;
|
||||||
|
}
|
||||||
setFailure(e instanceof Error ? e.message : 'Не удалось выполнить вход');
|
setFailure(e instanceof Error ? e.message : 'Не удалось выполнить вход');
|
||||||
} finally {
|
} finally {
|
||||||
setSubmitting(false);
|
setSubmitting(false);
|
||||||
@ -41,8 +47,8 @@ function DevLoginForm({ onLogin, error }: Props) {
|
|||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
// Production: вход выполняет внешний SSO; если загрузка профиля не удалась, показываем причину и даём повторить запрос
|
// Dev-эмитент выключен: вход выполняет внешний SSO; если загрузка профиля не удалась, показываем причину и даём повторить запрос
|
||||||
function ProdNotice({ onLogin, error }: Props) {
|
function SsoNotice({ onLogin, error }: Props) {
|
||||||
return (
|
return (
|
||||||
<div style={{ width: 360 }}>
|
<div style={{ width: 360 }}>
|
||||||
<p>Вход выполняется через РАИСА</p>
|
<p>Вход выполняется через РАИСА</p>
|
||||||
@ -54,15 +60,22 @@ function ProdNotice({ onLogin, error }: Props) {
|
|||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Какую форму показать, решает не режим сборки, а бэкенд во время выполнения: платформа всегда собирает фронт
|
||||||
|
* в production, при этом контур может работать на dev-заглушке аутентификации (переменные сборки в пайплайне
|
||||||
|
* задать нельзя). Поэтому форма входа по email показывается всегда, а заглушка про SSO — только после того,
|
||||||
|
* как бэкенд ответил, что маршрута /dev/token нет.
|
||||||
|
*/
|
||||||
export function DevLogin(props: Props) {
|
export function DevLogin(props: Props) {
|
||||||
|
const [ssoOnly, setSsoOnly] = useState(false);
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<div style={{ display: 'flex', justifyContent: 'center', alignItems: 'center', minHeight: '100svh' }}>
|
<div style={{ display: 'flex', justifyContent: 'center', alignItems: 'center', minHeight: '100svh' }}>
|
||||||
{/* Литерал process.env.NODE_ENV rsbuild подставляет при сборке: в production форма вырезается из бандла целиком */}
|
{ssoOnly ? (
|
||||||
{process.env.NODE_ENV !== 'production' ? (
|
// TODO(OQ-54): параметры JWT РАИСА (издатель, claim с email) и сам сценарий входа через SSO
|
||||||
<DevLoginForm {...props} />
|
<SsoNotice {...props} />
|
||||||
) : (
|
) : (
|
||||||
// TODO(OQ): параметры JWT РАИСА (издатель, claim с email)
|
<DevLoginForm {...props} onDisabled={() => setSsoOnly(true)} />
|
||||||
<ProdNotice {...props} />
|
|
||||||
)}
|
)}
|
||||||
</div>
|
</div>
|
||||||
);
|
);
|
||||||
|
|||||||
20
web/src/auth/devIssuer.test.ts
Normal file
20
web/src/auth/devIssuer.test.ts
Normal file
@ -0,0 +1,20 @@
|
|||||||
|
import { describe, expect, it } from 'vitest';
|
||||||
|
import { ApiError } from '../api/client';
|
||||||
|
import { isDevIssuerDisabled } from './devIssuer';
|
||||||
|
|
||||||
|
describe('isDevIssuerDisabled', () => {
|
||||||
|
it('404 «Not Found» без кода — маршрута /dev/token нет, dev-эмитент выключен', () => {
|
||||||
|
expect(isDevIssuerDisabled(new ApiError(404, undefined, 'Not Found', { detail: 'Not Found' }))).toBe(true);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('404 «пользователь не найден или отключён» — маршрут есть, это ошибка входа', () => {
|
||||||
|
expect(isDevIssuerDisabled(new ApiError(404, undefined, 'пользователь не найден или отключён'))).toBe(false);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('другие статусы и не-ApiError не считаются выключенным эмитентом', () => {
|
||||||
|
expect(isDevIssuerDisabled(new ApiError(409, undefined, 'неоднозначная учётная запись'))).toBe(false);
|
||||||
|
expect(isDevIssuerDisabled(new ApiError(0, 'NETWORK', 'Нет связи с сервером'))).toBe(false);
|
||||||
|
expect(isDevIssuerDisabled(new Error('Not Found'))).toBe(false);
|
||||||
|
expect(isDevIssuerDisabled(undefined)).toBe(false);
|
||||||
|
});
|
||||||
|
});
|
||||||
10
web/src/auth/devIssuer.ts
Normal file
10
web/src/auth/devIssuer.ts
Normal file
@ -0,0 +1,10 @@
|
|||||||
|
import { ApiError } from '../api/client';
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Маршрут `POST /dev/token` существует только при `DEBUG=true` и `AUTH_DEV_ISSUER=true` на бэкенде; иначе приложение
|
||||||
|
* отвечает 404 со строкой «Not Found» (а не доменной ошибкой с кодом). Это и есть признак «dev-эмитент выключен,
|
||||||
|
* вход только через SSO». 404 «пользователь не найден или отключён» приходит с того же статуса, но от самого маршрута.
|
||||||
|
*/
|
||||||
|
export function isDevIssuerDisabled(error: unknown): boolean {
|
||||||
|
return error instanceof ApiError && error.status === 404 && error.code === undefined && error.message === 'Not Found';
|
||||||
|
}
|
||||||
Loading…
x
Reference in New Issue
Block a user