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.

Sign up for GitHub

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

Jump to bottom

Legacy cipher 지원 종료를 위한 TLS 지원 범위 문서 추가 #294

Merged
merged 17 commits into from
Mar 26, 2024
Merged

Legacy cipher 지원 종료를 위한 TLS 지원 범위 문서 추가 #294

merged 17 commits into from
Mar 26, 2024

Conversation

SonnySon17
Copy link
Contributor

@SonnySon17 SonnySon17 commented Jan 15, 2024 •
edited by simnalamburt
Loading

  • Content 내용 관련 검수 및 피드백 필요 with @simnalamburt
  • 문서 위치 및 구성 관련 논의 및 변경 필요 with @sso-ashley

@SonnySon17 SonnySon17 self-assigned this Jan 15, 2024
@vercel Vercel
Copy link

vercel bot commented Jan 15, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
developers ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 26, 2024 5:43am

@simnalamburt
Copy link
Contributor

렌더된 문서 보러가기: https://developers-git-docs-add-legacy-cipher-retirement-guide-portone.vercel.app/docs/ko/tip/tls-support?v=v1

simnalamburt
simnalamburt reviewed Jan 15, 2024
View reviewed changes
Copy link
Contributor

@simnalamburt simnalamburt left a comment

Choose a reason for hiding this comment

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

표에서 cipher suite부분 글자크기를 두배이상 줄여서, 표의 전체적인 생김새가 한눈에 보이도록 바뀌면 좋을거같습니다

image

아직 보다말았는데 더 리뷰하겠습니다

src/content/docs/ko/tip/tls-support.mdx Outdated Show resolved Hide resolved
src/content/docs/ko/tip/tls-support.mdx Outdated Show resolved Hide resolved
@SonnySon17
Copy link
Contributor Author

image
@simnalamburt 2배이상으로 줄이면 너무 가시성이 떨어져서 사이즈를 60%로 줄이고 table width도 60%로 조정했습니다.

@SonnySon17
Copy link
Contributor Author

@disjukr 리을 mdx에서 스타일이 적용되어야하는 테이블이 필요한데 제가 F/E 및 mdx 사용 경험이 별로 없어서 일단 페이지 안에 자체 table을 html로 선언해놓고 css파일을 assets 폴더에 따로 선언해놓았습니다. 혹시 더 적합한 사용 방법이 있을까요?

@disjukr
Copy link
Contributor

disjukr commented Jan 18, 2024

@disjukr 리을 mdx에서 스타일이 적용되어야하는 테이블이 필요한데 제가 F/E 및 mdx 사용 경험이 별로 없어서 일단 페이지 안에 자체 table을 html로 선언해놓고 css파일을 assets 폴더에 따로 선언해놓았습니다. 혹시 더 적합한 사용 방법이 있을까요?

@SonnySon17 셀 병합기능까지 사용하는 복잡한 테이블이 필요한 경우에는 작업해주신 것처럼 html을 직접 작성하는 것 밖에 다른 도리가 없습니다.
스타일은 클래스명에 tailwind의 클래스명을 사용할 수 있기는 합니다만 지금 작업해주신 방향이 기술중립적인 면에서 좋아보입니다.

@sso-ashley
Copy link
Contributor

@SonnySon17 혹시 맨 위에 제목열에 배경색 넣을 수 있나요? 노션처럼 회색이라던지.. 볼드만으로는 잘 구분이 안되는것 같아서요. 그리고 혹시 일부러 왼쪽 정렬 하신게 아니라면 Chipher Suite 내용들 빼고는 가운데 정렬이 어떨까요?

@sso-ashley
Copy link
Contributor

  • HTTP 평문 통신 지원 중단
  • TLS 1.0, 1.1 지원 중단
  • 특정 조건을 만족하지 않는 legacy cipher suite에 대한 지원 중단

@SonnySon17 요약 내용이 표위로 올라오면 좋을 것 같습니다!

@sso-ashley
Copy link
Contributor

@SonnySon17 다른 페이지들은 반응형으로 모바일에서도 잘 보이는데 이것도 그렇게 해주실 수 있나요? 모바일에서 보면 오른쪽으로 스크롤해서 봐야하네요.. ㅠㅠ

@SonnySon17 SonnySon17 force-pushed the docs/add-legacy-cipher-retirement-guide branch from 29d363b to 6a88941 Compare January 25, 2024 10:20
@SonnySon17 SonnySon17 marked this pull request as ready for review January 25, 2024 10:21
@SonnySon17
Copy link
Contributor Author

@sso-ashley 요약 내용은 표 위로 올렸습니다.

제목열에 배경색 넣는거나 텍스트 정렬은 일반 mdx 문법에서는 지원을 안하고 좀 복잡하게 처리를 해야할 것 같은데 다른 문서들이랑 일관성을 좀 깨더라도 그렇게 할까요...? 지금 개발자센터에 있는 모든 문서들이 같은 템플릿(제목열에 배경색 없음, 왼쪽 정렬)이라 얘만 다르면 조금 이질적이지 않을까 싶어서요.

그리고 반응형 문제는 Java6용 가이드 중간에 있는 코드블럭들 때문인데 얘는 반응형으로 줄바꿈이 되면 사실상 읽기가 불가능해지는 놈이라 코드블럭 내부에서 스크롤 할 수 있게 만들 방법이 있는지 찾아보는중입니다

sso-ashley
sso-ashley reviewed Jan 25, 2024
View reviewed changes
Copy link
Contributor

@sso-ashley sso-ashley left a comment

Choose a reason for hiding this comment

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

@SonnySon17 제가 수정하려고 손을 대긴했는데.. 이거 문장 좀 다듬어 주실 수 있나요... 너무 논문같은 느낌이 들어가지구요.....

src/content/docs/ko/tip/tls-support.mdx Outdated Show resolved Hide resolved
src/content/docs/ko/tip/tls-support.mdx Outdated Show resolved Hide resolved
src/content/docs/ko/tip/tls-support.mdx Outdated Show resolved Hide resolved
src/content/docs/ko/tip/tls-support.mdx Outdated Show resolved Hide resolved
src/content/docs/ko/tip/tls-support.mdx Outdated Show resolved Hide resolved
src/content/docs/ko/tip/tls-support.mdx Outdated Show resolved Hide resolved
src/content/docs/ko/tip/tls-support.mdx Outdated Show resolved Hide resolved
src/content/docs/ko/tip/tls-support.mdx Outdated Show resolved Hide resolved
src/content/docs/ko/tip/tls-support.mdx Outdated Show resolved Hide resolved
src/content/docs/ko/tip/tls-support.mdx Outdated Show resolved Hide resolved
@sso-ashley
Copy link
Contributor

@SonnySon17 Java 6, 7, 8 하위의 1,2,3 메뉴도 접기로 넣어주실 수 있나요?

@simnalamburt simnalamburt force-pushed the docs/add-legacy-cipher-retirement-guide branch from f7e493e to 57d74eb Compare March 8, 2024 07:42
@simnalamburt simnalamburt force-pushed the docs/add-legacy-cipher-retirement-guide branch from 57d74eb to c81f64c Compare March 12, 2024 07:22
@simnalamburt
Copy link
Contributor

화면이 안나오네여

image

@sso-ashley
Copy link
Contributor

@SonnySon17 이거 린트 깨진것도 수정해주세요

@simnalamburt
Copy link
Contributor

I'm still working on this

before after
image image

@simnalamburt simnalamburt force-pushed the docs/add-legacy-cipher-retirement-guide branch from 1cd6c62 to ec31cb8 Compare March 25, 2024 15:47
@simnalamburt simnalamburt removed the request for review from disjukr March 25, 2024 15:48
@simnalamburt
Copy link
Contributor

I've made some major changes to the content, please check it out. @SonnySon17 @XiNiHa

@simnalamburt simnalamburt force-pushed the docs/add-legacy-cipher-retirement-guide branch from ec31cb8 to 6995ed1 Compare March 25, 2024 15:54
XiNiHa
XiNiHa approved these changes Mar 26, 2024
View reviewed changes
src/content/docs/ko/tip/tls-support.mdx Outdated Show resolved Hide resolved
src/content/docs/ko/tip/tls-support.mdx Outdated Show resolved Hide resolved
src/content/docs/ko/tip/tls-support.mdx Outdated Show resolved Hide resolved
src/content/docs/ko/tip/tls-support.mdx Outdated Show resolved Hide resolved
@simnalamburt simnalamburt merged commit cf9aa29 into main Mar 26, 2024
4 checks passed
@simnalamburt simnalamburt deleted the docs/add-legacy-cipher-retirement-guide branch March 26, 2024 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sso-ashley sso-ashley sso-ashley left review comments

@simnalamburt simnalamburt simnalamburt left review comments

@XiNiHa XiNiHa XiNiHa approved these changes

Assignees

@SonnySon17 SonnySon17

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@SonnySon17 @simnalamburt @disjukr @sso-ashley @XiNiHa