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

https로 실행되도록 설정 및 스크립트 명령어 추가 #632

Merged
merged 2 commits into from
Sep 19, 2023

Conversation

Gilpop8663
Copy link
Collaborator

🔥 연관 이슈

close: #631

📝 작업 요약

  • https로 실행되도록 설정 및 스크립트 명령어 추가

⏰ 소요 시간

30분

🔎 작업 상세 설명

  • https로 실행되도록 설정 및 스크립트 명령어 추가

🌟 논의 사항

https로 설정하면서 블로그에 정리해두었습니다

https://hell-of-company-builder.tistory.com/387

@github-actions
Copy link

github-actions bot commented Sep 17, 2023

⚡️ Lighthouse report!

Category Score
🔴 Performance 49
🟠 Accessibilty 88
🟠 SEO 85
🟠 PWA 89
Category Score
🟢 First Contentful Paint 0.7 s
🔴 Largest Contentful Paint 5.0 s
🔴 Total Blocking Time 3,340 ms
🟢 Cumulative Layout Shift 0
🟠 Speed Index 4.7 s

@@ -0,0 +1,28 @@
-----BEGIN PRIVATE KEY-----
Copy link
Member

Choose a reason for hiding this comment

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

pem 키는 보안적으로 굉장히 중요해서 github에는 올라가면 안될거 같아요!! .env 처럼 gitignore에 추가하는 것이 좋겠네요🙂

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

알려주셔서 감사합니다. 로컬에서만 사용되는 pem 키라서 gitignore를 해야할까 싶어서 다른 컴퓨터에서도 편히 사용하도록 파일을 올렸는데요. 팸키를 발급받은 기기에서만 유용한거라고 하네요 ! 새로운 사실을 알게 되었어요 👍

깃 이그노어에 추가하도록 할게요~

image

https://hojun-dev.tistory.com/entry/React-localhost-https-%EC%84%A4%EC%A0%95%ED%95%98%EA%B8%B0

발급받은 기기에서만 유효한 pem키이기 때문에 git에 올릴 필요가 없음
Copy link
Collaborator

@chsua chsua left a comment

Choose a reason for hiding this comment

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

이제 로컬에서도 https가 된다니 너무 좋아요!
생각보다 보안관련해서 http로 인해 할 수 없던 것들이 있는데
많이 개선될 것 같습니다!!
감사합니다! 고생하셨어요!

@chsua
Copy link
Collaborator

chsua commented Sep 19, 2023

fe-리뷰완

@Gilpop8663 Gilpop8663 merged commit 604625e into dev Sep 19, 2023
1 check passed
@Gilpop8663 Gilpop8663 deleted the feat/#631 branch September 19, 2023 04:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

https 환경으로 localhost 실행되는 코드 추가
3 participants