fix: SolapiMessageService에서 커스텀 PSR-18 HTTP 클라이언트 주입이 불가능하던 문제 수정#24
fix: SolapiMessageService에서 커스텀 PSR-18 HTTP 클라이언트 주입이 불가능하던 문제 수정#24Palbahngmiyine merged 1 commit intosolapi:masterfrom
Conversation
Summary of ChangesHello @KnifeLemon, 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! 이 풀 리퀘스트는 SolapiMessageService에서 사용자 정의 PSR-18 HTTP 클라이언트를 사용할 수 있도록 하여 서비스의 유연성을 향상시킵니다. 기존에는 내부적으로 고정된 HTTP 클라이언트 구현을 사용했지만, 이제 외부에서 ClientInterface를 구현한 클라이언트를 주입할 수 있게 되어, 개발자가 원하는 HTTP 클라이언트를 선택하거나 테스트 환경에서 모의 객체를 쉽게 사용할 수 있게 됩니다. Highlights
Changelog
Activity
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
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
|
안녕하세요 @KnifeLemon 님, 솔라피 기술지원팀입니다. 올리신 PR 기여에 대해서는 감사드리나, 해당 PR을 올리시게 된 상황등을 설명란에 명시해주시기 바라겠습니다. |
안녕하세요. 업데이트 이후 지속적으로 오류가 발생하였고, 출시노트(https://github.com/orgs/solapi/discussions/125)를 통해 하지만 제가 사용 중인 호스팅 환경(닷홈)의 경우 이에 따라 Guzzle 클라이언트를 직접 주입하는 방식으로 해결하고자 했습니다. 따라서 SolapiMessageService에서도 Custom Http Cient를 주입할 수 있도록 생성자 매개변수를 확장하였습니다. 물론 사용자 측에서 override를 통해 해결할 수도 있겠지만, 해당 패키지를 사용하는 입장에서 보면 직접 override를 구현해야 하는 구조보다는, 공식적으로 주입을 지원하는 편이 더 친절한 설계라고 생각하여 PR을 작성하게 되었습니다. 해당 변경은 기존 사용 방식과의 하위 호환성을 유지하면서 확장성을 제공하는 방향으로 작성하였습니다. 검토 부탁드립니다. 감사합니다. |
|
네 @KnifeLemon 님, 말씀 해 주신 사항 확인 했습니다, 해당 건의 경우 본래 의도라면 기여 코드와 같이 메시지 서비스에서 주입해주어야 하나, 테스트 부족으로 해당 사항이 빠진 것 같습니다. 기여해주신 코드는 큰 문제는 없는 것으로 보여 머지 하겠습니다. 다만 실제 릴리즈는 이번 주 내로 5.1.2 버전으로 진행될 예정인 점 참고 부탁드리겠습니다. |
2 participants
No description provided.