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 /spider/ prefix for REST APIs #139

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

Suggestion: Add /spider/ prefix for REST APIs #139

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

Comments

@jihoon-seo
Copy link
Member

As-Is:

  • /publicip
  • /vnetwork
  • /vm
  • ...

To-Be:

  • /spider/publicip
  • /spider/vnetwork
  • /spider/vm
  • ...

[사유]

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

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

@powerkimhub
Copy link
Member

@powerkimhub
Copy link
Member

completed.

ref) 3f55a94
ref) 246b169

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

2 participants