[마켓] 주문창 gRPC 통신 및 매수, 매도 주문 등록 #89

tongticktack · 2024-11-14T13:21:46Z

📚 이슈 번호

✅ 작업 내용

🔥 트러블 슈팅

0. gRPC IDL 인터페이스와 DTO 간 필드명 불일치

order.proto을 DTO와 일치하도록 수정해 해결

1. Prisma가 MongoDB replica set 환경에서 primary 노드에 제대로 연결되지 않음

Prisma가 dockerId을 읽어들이지 못해서 생긴 문제로, replica set config을 localhost로 설정해 해결

rs.reconfig({
  _id: "rs0",
  members: [
    { _id: 0, host: "localhost:27017" }
  ]
})

2. prisma schema database url

공통 모듈인 �Prisma Service 내에서 database의 url을 DATABASE_URL로 처리하고 있어 해당 환경변수를 불어오지 않아 생긴 문제였습니다.

datasource db {
  provider = "mongodb"
  url      = "mongodb:// ... " // 에러
}

🖋️ 학습 자료

gRPC
nestjs gRPC 학습정리

🐝 비고

Todo: transaction의 buy_order, sell_order 트랜젝션 실패시, gRPC로 balance 측 트랜젝션 롤백

Co-authored-by: Jeongwisdom <[email protected]>

Co-authored-by: Jeongwisdom [email protected]

- port# : 5001 Co-authored-by: Jeongwisdom [email protected]

Co-authored-by: Jeongwisdom <[email protected]>

- number -> string 정상화

- OrderLimitRequestDto: 지정가 매수, 매도 요청 - coin_code -> coinCode - order.proto 필드명 변경 - orderId -> historyId

- AuthenticatedGuard import문 경로 변경

- 기존 비어있는 테스트 파일 삭제 - 매수 매도 주문 - 주문 성공 - 잔액 부족시 BadRequestException - 트랜잭션 오류시InternalServerErrorException

tongticktack · 2024-11-14T13:26:04Z

@TaeRaeKim
auth에 쓰인 session모듈을 보고 따라서 transaction에 적용했습니다. 30a73e9
제대로 한건지 확인 부탁드립니다 🙏
package.json에 grpc moduleWrapper을 추가했습니다. 30a73e9

tongticktack · 2024-11-14T13:26:45Z

리뷰 후에 머지하면서 issue close 하도록 하겠습니다!

TaeRaeKim · 2024-11-14T13:40:15Z

prisma.schema작성 시, database의 url을 하드코딩하게 되면 << 하드코딩한 것 때문에 안된것이아니라
PrismaService 내에서 환경변수를 불러온 뒤 해당 DATABASE_URL로 db connection 생성하고 있어서 문제가 발생했던 것으로 기억합니다ㅎㅎ

TaeRaeKim · 2024-11-14T13:46:34Z

apps/balance/src/balance.controller.ts

+  @GrpcMethod('OrderService', 'makeBuyOrder')
+  async makeBuyOrder(buyOrderRequest: OrderRequestDto): Promise<OrderResponseDto> {
+    return await this.balanceService.makeBuyOrder(buyOrderRequest);
+  }


Grpc 전용 메소드들이 controller에 생성된 이유가 있을까요?

제 생각에는 service에 위치하는게 좋아 보입니다.

Router (Controller)는 사용자와 서버간에 상호작용하는 어플리케이션 계층에 속하는데, 비지니스 계층(TransactionService)에서 어플리케이션 계층(Balance Controller)를 호출하는 것이 적절치 않아보입니다.

동의합니다. 서비스 게층 간의 상호작용으로 보는게 더 적절하겠네요. 내용 반영하여 수정하겠습니다.

apps/transaction/src/main.ts

TaeRaeKim

고생하셨습니다!!

TaeRaeKim · 2024-11-14T13:56:14Z

apps/transaction/src/transaction.module.ts

+        options: {
+          package: 'order',
+          protoPath: '@app/grpc/order.proto',
+          url: 'localhost:5001',


localhost 보다는 실제로 필요한 데이터를 환경변수 형태로 넣어주세요!
ex) ${BALANCE_SERVICE_HOST}:${GRPC_PORT}

환경변수는 docker build 시에 주입해주고 있나요? 환경변수를 어디에 정의해야 적절할지요?

TaeRaeKim · 2024-11-14T13:57:25Z

nest-cli.json

@@ -60,6 +60,17 @@
      "compilerOptions": {
        "tsConfigPath": "libs/session/tsconfig.lib.json"
      }
+    },
+    "grpc": {


요기는 궁금한 내용인데 직접 작성해주신건가요? 아니면 nest-cli로 generate하면서 자동생성된 코드인가요?

배포 시, order.proto 파일을 카피하기 위해 직접 넣어준 코드입니다.

apps/balance/src/balance.module.ts

TaeRaeKim · 2024-11-14T13:59:52Z

package.json

@@ -105,7 +105,8 @@
      "^@app/common(|/.*)$": "<rootDir>/libs/common/src/$1",
      "^@app/prisma(|/.*)$": "<rootDir>/libs/prisma/src/$1",
      "^@app/session(|/.*)$": "<rootDir>/libs/session/src/$1",
-      "^@app/redis(|/.*)$": "<rootDir>/libs/redis/src/$1"
+      "^@app/redis(|/.*)$": "<rootDir>/libs/redis/src/$1",


레디스모듈은 잘못추가된거같네요~

"^@app/redis(|/.*)$": "<rootDir>/libs/redis/src/$1",

그럼 요 라인은 필요 없고 삭제하면 될까요?

TaeRaeKim · 2024-11-14T14:00:59Z

apps/transaction/src/transaction.module.ts

 import { PrismaModule } from '@app/prisma';
 import { ClientsModule, Transport } from '@nestjs/microservices';
 import { TransactionGrcpService } from './transaction.grcp.service';
+import { SessionModule } from '@app/session';



오 여기서는 고쳐졌군여 ㅎㅎ 앞에서 리뷰를 달았는데 머쓱하네요

TaeRaeKim · 2024-11-14T14:38:32Z

transaction grpc service를 transactionOrderService로 이름 변경해주세요.
transaction grpc에 있는 interface OrderService 를 libs/grpc 로 옮겨주세요
(transactionOrderService, BalanceOrderService에서 impl 해주세요)
transactionOrderService내에서 await firstValueFrom() 작업을 해주세요.
balance.controller 내부의 grpc 관련 코드를 balanceOrderService를 만들고 그곳으로 이동시켜주세요.

tongticktack · 2024-11-14T15:31:23Z

prisma.schema작성 시, database의 url을 하드코딩하게 되면 << 하드코딩한 것 때문에 안된것이아니라 PrismaService 내에서 환경변수를 불러온 뒤 해당 DATABASE_URL로 db connection 생성하고 있어서 문제가 발생했던 것으로 기억합니다ㅎㅎ

앗 좋은 지적 감사합니다

tongticktack and others added 10 commits November 14, 2024 18:00

✨Feat. orderHistory, sellOdrer, buyOrder 모델링

a58a60e

Co-authored-by: Jeongwisdom <[email protected]>

✨Feat. 주문 요청, 응답 dto 작성

ed41094

Co-authored-by: Jeongwisdom [email protected]

✨Feat. asset gRPC 서버 구현 #74 #75 #76 #77 #78 #79 #80

a81d160

- port# : 5001 Co-authored-by: Jeongwisdom [email protected]

✨Feat. Transanction 매수, 매도 API, gRPC client 구현

54e41f8

Co-authored-by: Jeongwisdom <[email protected]>

⚙️Config. grpc: nest-cli 수정

5e6ab88

🐛Fix. userId 타입 에러, proto file 경로 수정

0968335

- number -> string 정상화

🐛Fix. 변수명 및 타입 변경, 테스트 쿼리 삭제

82b33ae

- OrderLimitRequestDto: 지정가 매수, 매도 요청 - coin_code -> coinCode - order.proto 필드명 변경 - orderId -> historyId

✨Feat. grpc moduleNameWrapper 추가

a517426

♻️Refactor. transaction; 세션 모듈 사용으로 변경

30a73e9

- AuthenticatedGuard import문 경로 변경

✨Feat. transaction controller 테스트 코드

a9c66d1

- 기존 비어있는 테스트 파일 삭제 - 매수 매도 주문 - 주문 성공 - 잔액 부족시 BadRequestException - 트랜잭션 오류시InternalServerErrorException

tongticktack requested review from TaeRaeKim and Jeongwisdom November 14, 2024 13:21

tongticktack assigned tongticktack and Jeongwisdom Nov 14, 2024

TaeRaeKim reviewed Nov 14, 2024

View reviewed changes

apps/transaction/src/main.ts Outdated Show resolved Hide resolved

TaeRaeKim reviewed Nov 14, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[마켓] 주문창 gRPC 통신 및 매수, 매도 주문 등록 #89

[마켓] 주문창 gRPC 통신 및 매수, 매도 주문 등록 #89

tongticktack commented Nov 14, 2024 •

edited

Loading

tongticktack commented Nov 14, 2024

tongticktack commented Nov 14, 2024

TaeRaeKim commented Nov 14, 2024

TaeRaeKim Nov 14, 2024

tongticktack Nov 14, 2024

TaeRaeKim left a comment

TaeRaeKim Nov 14, 2024

tongticktack Nov 14, 2024

TaeRaeKim Nov 14, 2024

tongticktack Nov 14, 2024

TaeRaeKim Nov 14, 2024

tongticktack Nov 14, 2024

TaeRaeKim Nov 14, 2024

TaeRaeKim commented Nov 14, 2024 •

edited

Loading

tongticktack commented Nov 14, 2024

[마켓] 주문창 gRPC 통신 및 매수, 매도 주문 등록 #89

Are you sure you want to change the base?

[마켓] 주문창 gRPC 통신 및 매수, 매도 주문 등록 #89

Conversation

tongticktack commented Nov 14, 2024 • edited Loading

📚 이슈 번호

✅ 작업 내용

🔥 트러블 슈팅

0. gRPC IDL 인터페이스와 DTO 간 필드명 불일치

1. Prisma가 MongoDB replica set 환경에서 primary 노드에 제대로 연결되지 않음

2. prisma schema database url

🖋️ 학습 자료

🐝 비고

tongticktack commented Nov 14, 2024

tongticktack commented Nov 14, 2024

TaeRaeKim commented Nov 14, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

TaeRaeKim left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

TaeRaeKim commented Nov 14, 2024 • edited Loading

tongticktack commented Nov 14, 2024

tongticktack commented Nov 14, 2024 •

edited

Loading

TaeRaeKim commented Nov 14, 2024 •

edited

Loading