-
Notifications
You must be signed in to change notification settings - Fork 1
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
[#6] 3.02 주식차트 정보 기능 구현 #32
Conversation
|
||
@Cron('*/5 9-16 * * 1-5') | ||
async cronStockIndexLists() { | ||
const stockLists = await Promise.all([ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🟡 여기 Promise.all로 한번에 비동기 처리하면 서버가 처음 켜졌을 때 기준으로 Access Token을 1분 이내로 4번 호출해서 403 에러 발생하지 않나요?
저는 403 에러 발생해서 다른 방식으로 수정했거든요.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
맞습니다ㅠㅠ 처음 켜지고 호출하면 에러 나더라고요. 진님께서 말씀하신 DB에 저장하는 방식으로 진행하면 이 부분 문제 없어지지 않을까요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DB에 저장하는 방식으로 해도 최초 access token 발급을 이런 로직으로 하려고 하면 문제가 생길 것 같긴 해요..
'Content-Type': 'application/json; charset=UTF-8', | ||
}, | ||
body: JSON.stringify({ | ||
grant_type: 'client_credentials', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🟢 저희 코드가 상당히 비슷할 거라 이런 grant_type을 상수로 한번에 관리하는 건 어떨까요?! 지난 학습 스프린트 때 멘토님께서 이런 매직 넘버는 상수로 관리하는게 좋다고 하셨거든요.
|
||
// interfaces | ||
|
||
interface AccessTokenInterface { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🟢 이후에 이런 interface 어디서 관리하실 건가요?
저도 지금 service 계층에 그냥 두고 사용 중인데 나중에 분리가 필요하지 않을까 싶어서 질문드립니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
앜 인터페이스 아마 각 도메인에 interface 디렉터리 두고 관리할 것 같아요! 추가로 파일 분리하면 자꾸 린트 오류 떠서 규칙도 수정해야할 것 같슴니당 참고자료
✅ 주요 작업
💭 고민과 해결과정