Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

/api/v1/auth/login SDK 관련 요청 사항 #25

Open
bkdragon0228 opened this issue Jul 9, 2023 · 0 comments
Open

/api/v1/auth/login SDK 관련 요청 사항 #25

bkdragon0228 opened this issue Jul 9, 2023 · 0 comments

Comments

@bkdragon0228
Copy link

bkdragon0228 commented Jul 9, 2023

아래는 로그인 SDK 내부 코드 입니다.

export declare function login(connection: IConnection, body: Primitive<login.Input>): Promise<login.Output>;
export declare namespace login {
    type Input = Primitive<LoginUserDto>;
    type Output = Primitive<Try<string>>;
    const METHOD: "POST";
    const PATH: string;
    const ENCRYPTED: Fetcher.IEncrypted;
    function path(): string;
}

login.out을 타입를 보면 Try으로 되어 있는데 이러면 isBusinessErrorGuard 로 에러객체를 잡을 수 없습니다.

다만 status : 401, statusText : Unauthorized 에러로 예외처리 해줄 수 있습니다만 다른 코드와 통일성을 위해 에러객체를 사용하는 것을 제안드립니다.

혹시 다른 의도나 생각이 있었다면 공유 부탁드립니다. 로그인이 실패하는 경우는 서버 문제가 아니라면 Unauthorized 뿐이니 일부러 에러객체 만드지 않은 거라는 생각도 드네요.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant