cloud-barista / cb-ladybug

Cloud-Barista Multi-Cloud Application Runtime Framework : Support Multi-Cloud Kubernetes Service
Apache License 2.0
11 stars 11 forks source link

HTTP GET Method RFC2731 Compliance Request #125

Closed joowons closed 2 years ago

joowons commented 2 years ago
powerkimhub commented 2 years ago

@joowons

jihoon-seo commented 2 years ago

이 이슈를 close한 PR #126 은 CB-MCKS가 CB-Tumblebug을 호출할 때 GET 호출의 불필요한 body 내용물을 없애는 PR로 보입니다.

한편, 이 이슈는 'CB-MCKS의 REST API 중에서 GET 이면서도 body를 받고 있는 것이 있다면 body를 안 받도록 변경하거나 QueryParam 으로 받는 방법도 추가하자' 라는 취지의 이슈로 보입니다.

따라서, 혹시 'CB-MCKS의 REST API 중에서 GET 이면서도 body를 받고 있는 것이 있다면' 이 이슈는 다시 오픈해야 할 것으로 보이는데요, @itnpeople 어떻게 생각하시나요? 😊

joowons commented 2 years ago

@jihoon-seo 제가 설명을 좀 부족하게 쓴것 같습니다. 확인한 결과 CB-MCKS의 REST API 에서 그런 경우는 없었고 찾아보니 CB-Tumblebug을 호출할 때 GET 사용시 불필요한 body 부분이 있어서 그 부분 반영한 PR입니다.