Skip to content

Conversation

@inoray
Copy link
Collaborator

@inoray inoray commented Dec 18, 2025

Checklist:

  • Documentation has been updated, if necessary.
  • Examples have been added, if necessary.
  • Tests have been added, if necessary.

@gemini-code-assist
Copy link

Summary of Changes

Hello @inoray, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 내부 모델 API의 기본 URL을 genos.mnc.ai에서 genos.genon.ai로 업데이트합니다. 이 변경은 시스템이 올바른 엔드포인트를 가리키도록 보장하여 서비스 연속성과 잠재적인 인프라 변경 사항을 지원합니다. 또한, 일부 파일에서 사소한 코드 스타일 정리가 이루어졌습니다.

Highlights

  • URL 업데이트: 내부 모델 API의 기본 URL을 genos.mnc.ai에서 genos.genon.ai로 변경했습니다.
  • 코드 정리: 삼성증권리서치센터.py 파일에서 불필요한 공백을 제거하여 코드 가독성을 개선했습니다.

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

@inoray inoray requested a review from mestanam-mnc December 18, 2025 07:03
Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

이 PR은 내부 모델 URL을 genos.mnc.ai에서 genos.genon.ai로 성공적으로 수정했습니다. 하지만 코드 리뷰 과정에서 여러 파일에 걸쳐 API 키와 URL이 하드코딩되어 있는 더 큰 문제를 발견했습니다. 이는 심각한 보안 위험을 초래할 수 있으며 유지보수를 어렵게 만듭니다. 이러한 민감한 정보와 설정값들은 코드에서 분리하여 환경 변수나 별도의 설정 파일을 통해 관리하는 것이 강력히 권장됩니다. 이렇게 리팩토링하면 보안이 강화되고 향후 설정 변경이 용이해질 것입니다.

Comment on lines +1199 to 1202
toc_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/502/v1/chat/completions",
metadata_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/502/v1/chat/completions",
toc_api_key="022653a3743849e299f19f19d323490b",
metadata_api_key="022653a3743849e299f19f19d323490b",

Choose a reason for hiding this comment

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

critical

API URL과 키가 코드에 직접 하드코딩되어 있습니다. 이는 다음과 같은 여러 문제를 야기합니다.

  • 보안 취약점: API 키와 같은 민감 정보가 코드에 포함되면 버전 관리 시스템을 통해 유출될 수 있으며, 이는 무단 접근으로 이어질 수 있습니다.
  • 유지보수 어려움: URL이나 키가 변경될 때마다 여러 파일의 코드를 직접 수정하고 재배포해야 합니다. 현재 이 설정 블록이 여러 파일에 중복되어 있어 문제가 더욱 심각합니다.

개선 제안:
이러한 설정값들은 환경 변수나 별도의 설정 파일(예: .env, config.py)로 분리하여 관리하는 것이 좋습니다. 이렇게 하면 보안이 강화되고 설정 변경이 용이해집니다.

예를 들어, os.getenv()를 사용하여 환경 변수에서 값을 읽어올 수 있습니다.

import os

# ...
toc_api_base_url=os.getenv("TOC_API_BASE_URL"),
metadata_api_base_url=os.getenv("METADATA_API_BASE_URL"),
toc_api_key=os.getenv("TOC_API_KEY"),
metadata_api_key=os.getenv("METADATA_API_KEY"),

또한, 중복되는 설정 블록을 중앙에서 관리하는 설정 클래스나 모듈로 리팩토링하는 것을 강력히 권장합니다.

Suggested change
toc_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/502/v1/chat/completions",
metadata_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/502/v1/chat/completions",
toc_api_key="022653a3743849e299f19f19d323490b",
metadata_api_key="022653a3743849e299f19f19d323490b",
toc_api_base_url=os.getenv("TOC_API_BASE_URL"),
metadata_api_base_url=os.getenv("METADATA_API_BASE_URL"),
toc_api_key=os.getenv("TOC_API_KEY"),
metadata_api_key=os.getenv("METADATA_API_KEY"),

Comment on lines +848 to 851
toc_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/502/v1/chat/completions",
metadata_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/502/v1/chat/completions",
toc_api_key="022653a3743849e299f19f19d323490b",
metadata_api_key="022653a3743849e299f19f19d323490b",

Choose a reason for hiding this comment

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

critical

API URL과 키가 코드에 직접 하드코딩되어 있습니다. 이는 다음과 같은 여러 문제를 야기합니다.

  • 보안 취약점: API 키와 같은 민감 정보가 코드에 포함되면 버전 관리 시스템을 통해 유출될 수 있으며, 이는 무단 접근으로 이어질 수 있습니다.
  • 유지보수 어려움: URL이나 키가 변경될 때마다 여러 파일의 코드를 직접 수정하고 재배포해야 합니다. 현재 이 설정 블록이 여러 파일에 중복되어 있어 문제가 더욱 심각합니다.

개선 제안:
이러한 설정값들은 환경 변수나 별도의 설정 파일(예: .env, config.py)로 분리하여 관리하는 것이 좋습니다. 이렇게 하면 보안이 강화되고 설정 변경이 용이해집니다.

예를 들어, os.getenv()를 사용하여 환경 변수에서 값을 읽어올 수 있습니다.

import os

# ...
toc_api_base_url=os.getenv("TOC_API_BASE_URL"),
metadata_api_base_url=os.getenv("METADATA_API_BASE_URL"),
toc_api_key=os.getenv("TOC_API_KEY"),
metadata_api_key=os.getenv("METADATA_API_KEY"),

또한, 중복되는 설정 블록을 중앙에서 관리하는 설정 클래스나 모듈로 리팩토링하는 것을 강력히 권장합니다.

Suggested change
toc_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/502/v1/chat/completions",
metadata_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/502/v1/chat/completions",
toc_api_key="022653a3743849e299f19f19d323490b",
metadata_api_key="022653a3743849e299f19f19d323490b",
toc_api_base_url=os.getenv("TOC_API_BASE_URL"),
metadata_api_base_url=os.getenv("METADATA_API_BASE_URL"),
toc_api_key=os.getenv("TOC_API_KEY"),
metadata_api_key=os.getenv("METADATA_API_KEY"),

Comment on lines +893 to 896
toc_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/502/v1/chat/completions",
metadata_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/502/v1/chat/completions",
toc_api_key="022653a3743849e299f19f19d323490b",
metadata_api_key="022653a3743849e299f19f19d323490b",

Choose a reason for hiding this comment

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

critical

API URL과 키가 코드에 직접 하드코딩되어 있습니다. 이는 다음과 같은 여러 문제를 야기합니다.

  • 보안 취약점: API 키와 같은 민감 정보가 코드에 포함되면 버전 관리 시스템을 통해 유출될 수 있으며, 이는 무단 접근으로 이어질 수 있습니다.
  • 유지보수 어려움: URL이나 키가 변경될 때마다 여러 파일의 코드를 직접 수정하고 재배포해야 합니다. 현재 이 설정 블록이 여러 파일에 중복되어 있어 문제가 더욱 심각합니다.

개선 제안:
이러한 설정값들은 환경 변수나 별도의 설정 파일(예: .env, config.py)로 분리하여 관리하는 것이 좋습니다. 이렇게 하면 보안이 강화되고 설정 변경이 용이해집니다.

예를 들어, os.getenv()를 사용하여 환경 변수에서 값을 읽어올 수 있습니다.

import os

# ...
toc_api_base_url=os.getenv("TOC_API_BASE_URL"),
metadata_api_base_url=os.getenv("METADATA_API_BASE_URL"),
toc_api_key=os.getenv("TOC_API_KEY"),
metadata_api_key=os.getenv("METADATA_API_KEY"),

또한, 중복되는 설정 블록을 중앙에서 관리하는 설정 클래스나 모듈로 리팩토링하는 것을 강력히 권장합니다.

Suggested change
toc_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/502/v1/chat/completions",
metadata_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/502/v1/chat/completions",
toc_api_key="022653a3743849e299f19f19d323490b",
metadata_api_key="022653a3743849e299f19f19d323490b",
toc_api_base_url=os.getenv("TOC_API_BASE_URL"),
metadata_api_base_url=os.getenv("METADATA_API_BASE_URL"),
toc_api_key=os.getenv("TOC_API_KEY"),
metadata_api_key=os.getenv("METADATA_API_KEY"),

Comment on lines +985 to 988
toc_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/364/v1/chat/completions",
metadata_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/364/v1/chat/completions",
toc_api_key="a2ffe48f40ab4cf9a0699deac1c0cb76",
metadata_api_key="a2ffe48f40ab4cf9a0699deac1c0cb76",

Choose a reason for hiding this comment

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

critical

API URL과 키가 코드에 직접 하드코딩되어 있습니다. 이는 다음과 같은 여러 문제를 야기합니다.

  • 보안 취약점: API 키와 같은 민감 정보가 코드에 포함되면 버전 관리 시스템을 통해 유출될 수 있으며, 이는 무단 접근으로 이어질 수 있습니다.
  • 유지보수 어려움: URL이나 키가 변경될 때마다 여러 파일의 코드를 직접 수정하고 재배포해야 합니다. 현재 이 설정 블록이 여러 파일에 중복되어 있어 문제가 더욱 심각합니다.

개선 제안:
이러한 설정값들은 환경 변수나 별도의 설정 파일(예: .env, config.py)로 분리하여 관리하는 것이 좋습니다. 이렇게 하면 보안이 강화되고 설정 변경이 용이해집니다.

예를 들어, os.getenv()를 사용하여 환경 변수에서 값을 읽어올 수 있습니다.

import os

# ...
toc_api_base_url=os.getenv("TOC_API_BASE_URL"),
metadata_api_base_url=os.getenv("METADATA_API_BASE_URL"),
toc_api_key=os.getenv("TOC_API_KEY"),
metadata_api_key=os.getenv("METADATA_API_KEY"),

또한, 중복되는 설정 블록을 중앙에서 관리하는 설정 클래스나 모듈로 리팩토링하는 것을 강력히 권장합니다.

Suggested change
toc_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/364/v1/chat/completions",
metadata_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/364/v1/chat/completions",
toc_api_key="a2ffe48f40ab4cf9a0699deac1c0cb76",
metadata_api_key="a2ffe48f40ab4cf9a0699deac1c0cb76",
toc_api_base_url=os.getenv("TOC_API_BASE_URL"),
metadata_api_base_url=os.getenv("METADATA_API_BASE_URL"),
toc_api_key=os.getenv("TOC_API_KEY"),
metadata_api_key=os.getenv("METADATA_API_KEY"),

Comment on lines +1116 to 1119

Choose a reason for hiding this comment

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

critical

API URL과 키가 코드에 직접 하드코딩되어 있습니다. 이는 다음과 같은 여러 문제를 야기합니다.

  • 보안 취약점: API 키와 같은 민감 정보가 코드에 포함되면 버전 관리 시스템을 통해 유출될 수 있으며, 이는 무단 접근으로 이어질 수 있습니다.
  • 유지보수 어려움: URL이나 키가 변경될 때마다 여러 파일의 코드를 직접 수정하고 재배포해야 합니다. 현재 이 설정 블록이 여러 파일에 중복되어 있어 문제가 더욱 심각합니다.

개선 제안:
이러한 설정값들은 환경 변수나 별도의 설정 파일(예: .env, config.py)로 분리하여 관리하는 것이 좋습니다. 이렇게 하면 보안이 강화되고 설정 변경이 용이해집니다.

예를 들어, os.getenv()를 사용하여 환경 변수에서 값을 읽어올 수 있습니다.

import os

# ...
toc_api_base_url=os.getenv("TOC_API_BASE_URL"),
metadata_api_base_url=os.getenv("METADATA_API_BASE_URL"),
toc_api_key=os.getenv("TOC_API_KEY"),
metadata_api_key=os.getenv("METADATA_API_KEY"),

또한, 중복되는 설정 블록을 중앙에서 관리하는 설정 클래스나 모듈로 리팩토링하는 것을 강력히 권장합니다.

Suggested change
toc_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/364/v1/chat/completions",
metadata_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/364/v1/chat/completions",
toc_api_key="a2ffe48f40ab4cf9a0699deac1c0cb76",
metadata_api_key="a2ffe48f40ab4cf9a0699deac1c0cb76",
toc_api_base_url=os.getenv("TOC_API_BASE_URL"),
metadata_api_base_url=os.getenv("METADATA_API_BASE_URL"),
toc_api_key=os.getenv("TOC_API_KEY"),
metadata_api_key=os.getenv("METADATA_API_KEY"),

Comment on lines +1069 to 1072
# toc_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/502/v1/chat/completions",
# metadata_api_base_url="https://genos.genon.ai:3443/api/gateway/rep/serving/502/v1/chat/completions",
# toc_api_key="022653a3743849e299f19f19d323490b",
# metadata_api_key="022653a3743849e299f19f19d323490b",

Choose a reason for hiding this comment

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

medium

주석 처리된 코드이지만, API URL과 키 같은 민감한 정보가 코드에 남아있는 것은 바람직하지 않습니다. 이 정보들은 나중에 실수로 활성화될 수 있으며, 코드베이스에 민감한 정보를 남겨두는 것 자체가 보안상 위험이 될 수 있습니다. 사용되지 않는 설정이라면 완전히 제거하고, 나중에 참고해야 한다면 외부 문서나 안전한 설정 관리 도구에 기록하는 것을 고려해 보세요.

@inoray inoray requested a review from JaeseungYang January 2, 2026 04:47
@inoray inoray merged commit 0839090 into develop Jan 2, 2026
2 checks passed
@inoray inoray deleted the task/136-modify-ocr-endpoint branch January 2, 2026 04:50
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.

3 participants