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

feat: プロファイル画像取得APIの実装 #257

Merged
merged 4 commits into from
Dec 28, 2024
Merged

Conversation

mt-osho-san
Copy link
Collaborator

@mt-osho-san mt-osho-san commented Dec 28, 2024

User description

概要

プロファイル画像を取得する API を作成

変更点

  • Lambda, API GW の CDK コードを追加
  • Lambda 関数の中身の Python コードを追加

影響範囲

既存の機能には影響なし

テスト

  • pytest によりテストを実行

関連 Issue


PR Type

Enhancement


Description

  • プロファイル画像を取得するAPIを新たに実装しました。
  • Lambda関数を追加し、S3から画像を取得してBase64エンコードした結果を返します。
  • API GatewayにGETメソッドを追加し、ユーザーのプロフィール画像を取得できるようにしました。
  • 画像取得関数に対するユニットテストを実装し、エラーハンドリングも確認しました。

Changes walkthrough 📝

Relevant files
Enhancement
settings.ts
プロファイル画像取得用LambdaとAPIの追加                                                                 

src/backend/lib/constructs/settings.ts

  • プロファイル画像取得用のLambda関数を追加
  • API GatewayにGETメソッドを追加
+15/-0   
get_profile_image.py
S3から画像を取得するLambda関数の実装                                                                     

src/backend/lambda/get_profile_image/get_profile_image.py

  • S3から画像を取得する関数を実装
  • 画像をBase64エンコードして返す処理を追加
+88/-0   
Tests
test_get_profile_image.py
プロファイル画像取得関数のテスト追加                                                                             

src/backend/test/pytest/test_get_profile_image.py

  • 画像取得関数のユニットテストを追加
  • エラーハンドリングのテストを実装
+65/-0   

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

@mt-osho-san mt-osho-san linked an issue Dec 28, 2024 that may be closed by this pull request
@mt-osho-san mt-osho-san self-assigned this Dec 28, 2024
@github-actions github-actions bot changed the title Feature/#244 feat: プロファイル画像取得APIの実装 Dec 28, 2024
Copy link
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅

244 - Fully compliant

Fully compliant requirements:

  • API の名前 : GET /profile/image
  • クエリパラメーター : なし (event から user_id を取得)
  • s3 の user-setting-bucket の /profile/
  • return は base64 エンコードした画像

Not compliant requirements:
No

⏱️ Estimated effort to review: 4 🔵🔵🔵🔵⚪
🔒 No security concerns identified
🔀 No multiple PR themes
⚡ Recommended focus areas for review

Possible Issue

content_type の取得時に ContentType ではなく ContentType であるべきです。これにより、正しいコンテンツタイプが取得できない可能性があります。

content_type = response["ContentType"]

@Kota8102 Kota8102 merged commit 8a2f23d into main Dec 28, 2024
12 checks passed
@Kota8102 Kota8102 deleted the feature/#244 branch December 28, 2024 08:29
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.

設定画面の画像を取ってくる API を作成
2 participants