Skip to content

Conversation

@Hyunsoo-ds
Copy link

5월 프론트엔드 포스팅입니다!

@hoqn hoqn requested review from custardcream98 and hoqn May 28, 2023 04:31
@hoqn hoqn added the 🙋‍♂️🙋‍♀️ 리뷰 요청 PR 리뷰를 요청합니다~ label May 28, 2023
Copy link

@hoqn hoqn left a comment

Choose a reason for hiding this comment

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

SOP, CORS와 같은 보안 부분은 프론트엔드 작업을 하면서 간과하기도 쉽고, 정확히 이해하기 어려운 부분이 있는데, 관련 내용을 쉽게 정리해주셔서 잘 읽었습니다!

내용이나 구성은 너무 좋았고, 형식 관련해서만 아래 몇 가지 수정해주시면 좋을 것 같아요

좋은 포스팅 감사합니다 :)


* ## **port**

* 위의 예시에서 ':1001' 해당함
Copy link

Choose a reason for hiding this comment

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

Suggested change
* 위의 예시에서 ':1001' 해당함
* 위의 예시에서 ':1001' 해당함

Comment on lines 33 to 35
오리진은 브라우저가 가져온 정보의 출처를 의미합니다. 그리고 오리진은 크게 프로토콜(Protocol, Scheme), 포트(Port), 호스트(Host)로 구성됩니다.

이 때 오리진의 구성요소가 모두 일치해야지 동일한 오리진으로 판정됩니다.
Copy link

Choose a reason for hiding this comment

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

통일성을 위해 오리진을 모두 영어로 바꾸거나, 뒷부분처럼 오리진(Origin)과 같은 방식으로 표기하면 더 좋을 것 같아요

@SSUHYUNKIM SSUHYUNKIM added the 🛠 수정 완료 수정 요청 사항이 모두 해결됐을 때 label Jul 7, 2023
@hoqn hoqn added 💯 최종 완료 모든 수정과 리뷰가 끝난 후 merge를 기다릴 때 and removed 🛠 수정 완료 수정 요청 사항이 모두 해결됐을 때 🙋‍♂️🙋‍♀️ 리뷰 요청 PR 리뷰를 요청합니다~ labels Jul 7, 2023
@SSUHYUNKIM SSUHYUNKIM merged commit 94a02d2 into GDSC-University-of-Seoul:master Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💯 최종 완료 모든 수정과 리뷰가 끝난 후 merge를 기다릴 때

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants