나의 첫 번째 Github 이슈

최대 1 분 소요


1. Motivation

친애하는 동료이자 친구인 @jskim1991이 spring-cloud-openfeign 프로젝트에 PR(Pull Request)을 넣었다는 이야기를 듣고 한동안 깃허브(Github) 이슈를 찾아. 한동안 퇴근하면 스프링(spring)에 등록된 이슈들을 읽어보면서 호시탐탐 PR을 노렸다. 해결할만한 버그나 이슈가 있는지 찾아보면서 지내다 보니 적당한 이슈를 하나 찾아냈다.

2. Pick the issue

버그 내용을 보아하니 라이브러리 내부에서 배열(array)을 toString 메서드를 통해 그대로 문자열로 변경한 것 같다.

https://github.com/spring-cloud/spring-cloud-openfeign/issues/256

3. Suggestion solution for issue

API 호출 로직 내부를 디버깅해서 문제가 발생하는 코드 위치와 이를 해결할 방법을 제시하였다.

  • 특정 클래스에서 Iterable 상속 여부만 체크하기 때문에 발생한 문제이다.
    • 배열 객체는 Iterable을 상속하지 않는다.
  • 배열을 별도로 처리하는 로직이 추가되면 큰 영향 없이 사용할 수 있을 것이라 생각했다.
https://github.com/spring-cloud/spring-cloud-openfeign/issues/256


Openfeign으로 이슈를 옮기라는 답변을 받았다.

https://github.com/spring-cloud/spring-cloud-openfeign/issues/256


Openfeign 프로젝트에 해당 이슈를 등록했다.

https://github.com/OpenFeign/feign/issues/1170


해당 이슈는 버그로 등록되고, 직접 해결하겠다는 답변을 받았다.

https://github.com/OpenFeign/feign/issues/1170

CLOSING

당시 몇 시간 동안 버그와 해결 방법까지 찾아 제시했지만, 아쉽게 PR을 직접 넣진 못했다. 해당 이슈를 분석하기 위해 디버깅해보면서 많은 공부가 되었지만, 아쉬움이 남았다.

TEST CODE REPOSITORY

REFERENCE

댓글남기기