Skip to content

Hotfix/log base url#174

Merged
ahn0min merged 3 commits intomainfrom
hotfix/log_baseURL
Dec 6, 2023
Merged

Hotfix/log base url#174
ahn0min merged 3 commits intomainfrom
hotfix/log_baseURL

Conversation

@ahn0min
Copy link

@ahn0min ahn0min commented Dec 6, 2023

[원인]

  • node.js 환경에서는 global.location.hostnameundefined 이기 때문에 예상과 다르게 AuthAPIInstancebaseURL이 개발서버 URL을 가지게 되었습니다.

[해결]

  • next.js api routes의 req.headers.host 를 조금 가공하여 AuthAPIInstance.defaulth.baseURL을 직접 변경하여 문제를 해결하였습니다.

@netlify
Copy link

netlify bot commented Dec 6, 2023

Deploy Preview for snowball-dev-1c33edfdfwsdlki ready!

Name Link
🔨 Latest commit 3a19494
🔍 Latest deploy log https://app.netlify.com/sites/snowball-dev-1c33edfdfwsdlki/deploys/65701a5a31de3d00084c7b88
😎 Deploy Preview https://deploy-preview-174--snowball-dev-1c33edfdfwsdlki.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ahn0min ahn0min requested a review from Whale2200d December 6, 2023 06:55
@ahn0min ahn0min self-assigned this Dec 6, 2023
@ahn0min ahn0min merged commit c1fc0ae into main Dec 6, 2023
@ahn0min ahn0min deleted the hotfix/log_baseURL branch December 6, 2023 06:55
@ahn0min ahn0min restored the hotfix/log_baseURL branch December 6, 2023 07:15
ahn0min pushed a commit that referenced this pull request Dec 6, 2023
* chore: next.js api routes 에서 실행되는 login 관련 콘솔 추적 (#173)

* Hotfix/log base url (#174)

* chore: next.js api routes 에서 실행되는 login 관련 콘솔 추적

* feat: AuthAPIInstance baseURL 변경 로직 추가

* refactor: amplitude init code 간소화 리팩토링

* Hotfix/log base url (#175)

* chore: next.js api routes 에서 실행되는 login 관련 콘솔 추적

* feat: AuthAPIInstance baseURL 변경 로직 추가

* refactor: amplitude init code 간소화 리팩토링

* fix: baseURL에서 '/api/v1/user/' 를 추가
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

Successfully merging this pull request may close these issues.

2 participants