Skip to content

Merge develop to master#215

Merged
CSE-Shaco merged 9 commits intomasterfrom
develop
Dec 9, 2025
Merged

Merge develop to master#215
CSE-Shaco merged 9 commits intomasterfrom
develop

Conversation

@CSE-Shaco
Copy link
Contributor

#️⃣ Related Issue

ex) #이슈번호, #이슈번호

📝 PR Description

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

Screenshots (선택)

스크린샷을 첨부해주세요

💬 Reviewer Request (선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요

CSE-Shaco and others added 9 commits December 9, 2025 06:26
*Total -- 18,938.58kb -> 15,073.44kb (20.41%)

/public/images/activity/buildwithai.png -- 1,096.15kb -> 327.24kb (70.15%)
/public/icons/logo/chatgpt.png -- 66.55kb -> 34.74kb (47.8%)
/public/images/homecoming/main_img.png -- 2,315.78kb -> 1,459.04kb (37%)
/public/images/homecoming/bg.png -- 749.01kb -> 612.02kb (18.29%)
/public/icons/logo/django.png -- 23.86kb -> 19.84kb (16.83%)
/public/images/activity/seminar_bg.png -- 14,648.43kb -> 12,584.85kb (14.09%)
/public/icons/logo/figma.png -- 38.80kb -> 35.71kb (7.97%)

Signed-off-by: ImgBotApp <[email protected]>
fix(homecoming): typo 수정
Added GOOGLE_MAPS_API_KEY to environment variables for develop and main branches.
fix(homecoming): 디버깅을 위한 코드 추가

console.log(
"MAPS KEY PREFIX:",
process.env.NEXT_PUBLIC_GOOGLE_MAPS_API_KEY?.slice(0, 10)

Check failure

Code scanning / CodeQL

Clear-text logging of sensitive information High

This logs sensitive data returned by
an access to NEXT_PUBLIC_GOOGLE_MAPS_API_KEY
as clear text.

Copilot Autofix

AI 3 months ago

To address this logging of sensitive information, all code that logs (or otherwise exposes) any part of the NEXT_PUBLIC_GOOGLE_MAPS_API_KEY variable should be removed, especially the console statement on line 227. Simply remove the console.log block (lines 225-228) from the HomecomingMap function in src/app/homecoming/page.jsx. No imports, method replacements, or variable substitutions are required—just deleting the logging code is sufficient. This change will prevent the accidental or intentional logging of any part of the sensitive API key.

Suggested changeset 1
src/app/homecoming/page.jsx

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/src/app/homecoming/page.jsx b/src/app/homecoming/page.jsx
--- a/src/app/homecoming/page.jsx
+++ b/src/app/homecoming/page.jsx
@@ -222,11 +222,8 @@
         googleMapsApiKey: process.env.NEXT_PUBLIC_GOOGLE_MAPS_API_KEY, id: 'homecoming-map-script',
     });
 
-    console.log(
-        "MAPS KEY PREFIX:",
-        process.env.NEXT_PUBLIC_GOOGLE_MAPS_API_KEY?.slice(0, 10)
-    );
 
+
     const center = {lat: 37.388493, lng: 126.639989};
 
     if (loadError) {
EOF
@@ -222,11 +222,8 @@
googleMapsApiKey: process.env.NEXT_PUBLIC_GOOGLE_MAPS_API_KEY, id: 'homecoming-map-script',
});

console.log(
"MAPS KEY PREFIX:",
process.env.NEXT_PUBLIC_GOOGLE_MAPS_API_KEY?.slice(0, 10)
);


const center = {lat: 37.388493, lng: 126.639989};

if (loadError) {
Copilot is powered by AI and may make mistakes. Always verify output.
Unable to commit as this autofix suggestion is now outdated
@CSE-Shaco CSE-Shaco merged commit 669103b into master Dec 9, 2025
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants