Skip to content
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

HTTP GET Method RFC2731 Compliance Request #125

Closed
joowons opened this issue Jan 19, 2022 · 3 comments · Fixed by #126
Closed

HTTP GET Method RFC2731 Compliance Request #125

joowons opened this issue Jan 19, 2022 · 3 comments · Fixed by #126
Labels
enhancement New feature or request

Comments

@joowons
Copy link
Contributor

joowons commented Jan 19, 2022

http get 요청 시 body 보내는 부분이 있다면 Query Parameter로 변경 요청.
예) ConnectionName := c.QueryParam("connectionName")

@joowons joowons added the enhancement New feature or request label Jan 19, 2022
@powerkimhub
Copy link
Member

@joowons

  • 혹시, 관련 API가 존재한다면 변경 보다는 추가 고려해주시면 좋을 듯합니다.
    • 사유: Backward Compatibility

@itnpeople itnpeople linked a pull request Jan 21, 2022 that will close this issue
@jihoon-seo
Copy link
Member

이 이슈를 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
Copy link
Contributor Author

joowons commented Jan 21, 2022

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants