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

[Feature] image resizer #138

Open
5 tasks
koreanddinghwan opened this issue Oct 10, 2023 · 12 comments
Open
5 tasks

[Feature] image resizer #138

koreanddinghwan opened this issue Oct 10, 2023 · 12 comments
Assignees
Labels
enhancement New feature or request feature feature todo

Comments

@koreanddinghwan
Copy link
Contributor

Description

  • 인트라 cdn이 불안정함에따라 별도의 image 저장 프로세스가 필요하게 되었습니다.
  • 별도의 lambda를 사용하는 것이 아닌, api 서버 자원을 그대로 사용합니다.(유휴 상태 자원이 많음)
  • S3 + sharp 를 사용합니다.

Todo

  • 이미 login절차에서 42 oauth 서버와 통신이 느린 상태인데, 이미지 resize 과정까지 포함하면 더 느려질 것. 어떻게 처리할 것인가?
  • 사용할 image resizer lib 선택
  • 저장할 이미지 형식, 비율, 크기(w,h) 지정
  • 서버 모듈 구조 선택(notification module이 이미 aws sdk와 묶여있음. 이와 분리할건지)
  • s3 앞에 api-gateway를 붙여서 CORS관리 할 것인지 선택
@JuneParkCode
Copy link
Member

아니 이걸 그새...제가 하려고했는데 ㅋㅋ.확인해보겠습니다~

@JuneParkCode
Copy link
Member

  1. 초기 이미지를 default 지정으로 해걀
  • 굳이 인트라 프로필 변환해서 들고올 필요는 없을 것 같습니다. 이미지 추가가 가능한 이상 default를 별도로 두고, 이후에 유저가 직접 추가할 수 있도록 해주면 좋을 듯 합니다.

@koreanddinghwan
Copy link
Contributor Author

스크린샷 2023-10-10 16 05 08 스크린샷 2023-10-10 16 05 36

원본 인트라 이미지 -> 300kb

160 * 120 , jpeg : 2.6kb
160 * 120 , webp : 1.3kb

@JuneParkCode
Copy link
Member

default -> 인트라 default 말고 그냥 빈 아바타 이미지임당

@JuneParkCode
Copy link
Member

S3 + sharp 를 사용합니다.
resizer lib 는 sharp 쓰는것 아닌가여?

@koreanddinghwan
Copy link
Contributor Author

S3 + sharp 를 사용합니다.
resizer lib 는 sharp 쓰는것 아닌가여?

쓰시고 싶은거 있음 사용하셔도됨다

@koreanddinghwan
Copy link
Contributor Author

default -> 인트라 default 말고 그냥 빈 아바타 이미지임당

그럼 default image는 프론트 서버에 두고, 서버에서 '' 이렇게 리턴되면 default image렌더링하시면될것같아요

@JuneParkCode
Copy link
Member

w, h 는 300px * 300px 안쪽이면 춤분합니다.

@koreanddinghwan
Copy link
Contributor Author

모듈 구조는 어떻게 할까요? upload 모듈 별도로 만들어두는게 낫지않나..

@koreanddinghwan
Copy link
Contributor Author

notification쪽에서 SES 따로 사용하고있는데, S3랑 SES 모듈 합쳐서 AWS 모듈로 만들어서 처리하는것도 어떤가싶기도하구

@JuneParkCode
Copy link
Member

AWS를 별도로 분리하고 그거 가져다쓰게 해야할 것 같네여 upload 는 당근 따로 두고...

@koreanddinghwan
Copy link
Contributor Author

AWS를 별도로 분리하고 그거 가져다쓰게 해야할 것 같네여 upload 는 당근 따로 두고...

#144

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feature feature todo
Projects
None yet
Development

No branches or pull requests

2 participants