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

[FE] 마크다운 경고 콘솔문제 해결 및 http-proxy-middleware 설정 #127

Merged
merged 5 commits into from
Aug 8, 2022

Conversation

kimyouknow
Copy link
Contributor

About

  1. proxy 방식 변경

  2. 마크다운 경고 콘솔 해결

Description

1. proxy 방식 변경

pakcge.json의 proxy로 cors 문제를 개발환경에서 프로싱해서 해결할 수 있습니다. 하지만 추후에 더 유연한 환경설정을 위해 다양한 기능을 제공하는 http-proxy-middleware 라이브러리로 변경했습니다.

Ref: Proxying API Requests in Development - CRA 공식문서

2. 마크다운 경고 콘솔 해결

CRA관련 pr과 관련 있는 에러였습니다.

정확한 원인은 파악하지 못했지만, 임시적인 해결책은 찾았습니다. 임시적인 해결책이어서 추후에 문제가 될 수도 있다고 생각합니다.

"scripts": {
    "start": "GENERATE_SOURCEMAP=false react-scripts start",
   ...
  },

Ref: nhn/tui.editor#2137 (comment)

@kimyouknow kimyouknow added Fix 수정 FE 프론트 관련 labels Aug 8, 2022
@kimyouknow kimyouknow requested a review from hoeun0723 August 8, 2022 12:57
@kimyouknow kimyouknow self-assigned this Aug 8, 2022
Copy link
Collaborator

@hoeun0723 hoeun0723 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다 ! 바로 최신화 시킬게요 !

@@ -37,6 +37,7 @@
"aws-sdk": "^2.1118.0",
"axios": "^0.25.0",
"cross-env": "^7.0.3",
"http-proxy-middleware": "^2.0.6",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

질문

이런 식으로 proxy 서버 연결 코드를 수정하였을 때 저번엔 오류가 났던거 같은데 이젠 잘 해결 된건가요 ??

@@ -0,0 +1,11 @@
const { createProxyMiddleware } = require('http-proxy-middleware');
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

proxy 서버 질문 관련

아하 이렇게 따로 주소를 입력해주면 해결 되는 문제였군요 ! 확인했습니다

@hoeun0723 hoeun0723 merged commit 48f1128 into develop Aug 8, 2022
@kimyouknow kimyouknow deleted the FE/md-editor branch August 29, 2022 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트 관련 Fix 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants