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

Suggestion: Add /tumblebug/ prefix for REST APIs #96

Closed
jihoon-seo opened this issue Mar 11, 2020 · 2 comments
Closed

Suggestion: Add /tumblebug/ prefix for REST APIs #96

jihoon-seo opened this issue Mar 11, 2020 · 2 comments
Assignees

Comments

@jihoon-seo
Copy link
Member

As-Is:

  • /ns
  • /ns/{ns_id}/mcis
  • ...

To-Be:

  • /tumblebug/ns
  • /tumblebug/ns/{ns_id}/mcis
  • ...

[사유]

  • API-GW 에서 지금은 각 API (/ns, /ns/{ns_id}/mcis, ...) 별로 rule 을 적고 있습니다.
  • 향후 http://api-gw:8000/tumblebug/* 로 오는 모든 요청을 http://cb-tumblebug:1323/* 로 보내고 싶습니다.
  • 예를 들어 API-GW 테스트 시 Tumblebug 에 API 요청할 때 1) Tumblebug 로 직접 보내기, 2) API-GW 로 보내기 를 계속 왔다갔다 해야 하는데, 이 때 Tumblebug API 도 /tumblebug/ (또는 /cb-tumblebug/) 로 시작하도록 통일되어 있으면, 테스트도 편리하고, 향후에도 편리할 것으로 예상됩니다.

  • 영향: Tumblebug API 를 이용하는 Dragonfly, Webtool 등도 Tumblebug API 변경에 대한 대응이 필요합니다.
  • 기타: Spider, Dragonfly, Webtool 등도 모두 이런 스타일로 갔으면 좋겠습니다.
@jihoon-seo
Copy link
Member Author

@jihoon-seo
Copy link
Member Author

Resolved by: #98

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

No branches or pull requests

1 participant