Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
100Gyeon
requested review from
hyesungoh,
Dongkyuuuu,
gutenLEE,
metacode22,
Orchemi,
sangbooom,
Seojunhwan,
sjoleee and
sumi-0011
as code owners
sangbooom
approved these changes
Sep 2, 2024
| ## Topic 38 우연에 맡기는 프로그래밍 | ||
|
|
||
| 행운과 우연한 성공에 의존하는 대신 '의도적으로 프로그래밍'해야 한다. | ||
| 왜 코드가 망가졌는지 모르는 까닭은 애초에 코드가 왜 잘 돌아가는지도 몰랐기 때문이다. |
There was a problem hiding this comment.
저는 코드리뷰 할 때 useEffect가 있다면 반드시 써야되는 상황인가부터 봅니다.. 우연에 맡기는 프로그래밍에 일조하는 녀석이기 때문에..
|
|
||
| #### 1. 공격 표면을 최소화하라. | ||
|
|
||
| 공격 표면 영역 : 공격자가 데이터를 입력/추출하거나 서비스를 실행시킬 수 있는 모든 접근 지점을 합한 것 |
There was a problem hiding this comment.
eval 사용 금지가 생각나네요.. (도대체 왜 만들었을까)
| > 회사 노션에 용어 사전 테이블 하나 만들어서 '국문, 영문, 설명, 결정 근거'를 작성하고 있어요. | ||
| > API, 코드 레벨에서 용어를 통일할 수 있다는 점이 좋은 것 같아요. | ||
|
|
||
| 의도를 제대로 표현하지 못하거나 오해를 부를 수 있거나 헷갈리는 이름을 발견했다면 고쳐야 한다. |
gutenLEE
approved these changes
Sep 2, 2024
|
|
||
| 반드시 팀의 모든 사람이 각 단어의 뜻을 알고 일관성 있게 사용해야 한다. | ||
| 특별한 의미가 있는 단어를 모두 모은 **프로젝트 용어 사전**을 만드는 방법도 있다. | ||
| > 회사 노션에 용어 사전 테이블 하나 만들어서 '국문, 영문, 설명, 결정 근거'를 작성하고 있어요. |
hyesungoh
approved these changes
Sep 2, 2024
Comment on lines
+139
to
+144
| > 보안 업데이트 보니까 저는 dependabot이 떠오르네요. | ||
| > 여러분은 dependabot을 잘 활용하고 계신가요? | ||
| > 제가 담당했던 레거시 프로젝트는 무려 3년 전에 axios를 설치한 채로 방치되었고, dependabot PR 마저도 close된 상황이었어요. | ||
| > major version upgrade가 필요해서 부담되긴 했지만, 아래와 같이 태스크를 생성해 개선했던 경험이 있습니다. | ||
| > | ||
| > <img src="https://github.com/user-attachments/assets/b0b21cac-27fe-4491-b82f-e32a445c6c8e" width=400 /> |
There was a problem hiding this comment.
저희는 디펜더봇 쓰고 있긴 한데 .. PR 올려주는 것만 머지하고 있어요 😢
저희도 axios 올렸어야 했는데, 메이저 올리는 건 사이드 이펙트 파악할 엄두가 안돼서 못했었는데 대단하세요 👏 👏 👏
글애서 저희는 0.28 대로 올렸는데, 0 점대 버전도 꾸준히 업데이트를 하더라고요 👀
There was a problem hiding this comment.
오 그렇네용 요건 몰랐는데 공유 감사합니다👍
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
여기에서 확인해 주시면 됩니다🤓