-
Notifications
You must be signed in to change notification settings - Fork 2
내부모델 url의 genos.mnc.ai를 genos.genon.ai 로 수정 #139
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
Conversation
Summary of ChangesHello @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을 Highlights
🧠 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 AssistThe 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
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 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
|
There was a problem hiding this 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이 하드코딩되어 있는 더 큰 문제를 발견했습니다. 이는 심각한 보안 위험을 초래할 수 있으며 유지보수를 어렵게 만듭니다. 이러한 민감한 정보와 설정값들은 코드에서 분리하여 환경 변수나 별도의 설정 파일을 통해 관리하는 것이 강력히 권장됩니다. 이렇게 리팩토링하면 보안이 강화되고 향후 설정 변경이 용이해질 것입니다.
| 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", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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"),또한, 중복되는 설정 블록을 중앙에서 관리하는 설정 클래스나 모듈로 리팩토링하는 것을 강력히 권장합니다.
| 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"), |
| 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", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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"),또한, 중복되는 설정 블록을 중앙에서 관리하는 설정 클래스나 모듈로 리팩토링하는 것을 강력히 권장합니다.
| 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"), |
| 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", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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"),또한, 중복되는 설정 블록을 중앙에서 관리하는 설정 클래스나 모듈로 리팩토링하는 것을 강력히 권장합니다.
| 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"), |
| 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", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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"),또한, 중복되는 설정 블록을 중앙에서 관리하는 설정 클래스나 모듈로 리팩토링하는 것을 강력히 권장합니다.
| 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"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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"),또한, 중복되는 설정 블록을 중앙에서 관리하는 설정 클래스나 모듈로 리팩토링하는 것을 강력히 권장합니다.
| 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"), |
| # 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", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Checklist: