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

Watchkeeper post to razeedash fails when URL rewriting is used #83

Open
thecheryl opened this issue Feb 7, 2020 · 1 comment
Open
Assignees
Labels
enhancement New feature or request

Comments

@thecheryl
Copy link

Describe the bug
When using URL rewriting to route requests to Razee, watchkeeper fails to post to razeedash because the API base path is hardcoded to /api/v2 here https://github.com/razee-io/Razeedash-api/blob/1fe88efc21a2ba9e46aa8938fa9604828c0b7358/app/routes/install/index.js#L30.

In the watchkeeper logs we see “POST heartbeat to razeedash failed” and this is because it’s trying to post to https://<host>/api/v2/clusters/... instead of the route we are using for razeedash-api which is https://<host>/api/collector/v2/clusters/....

To Reproduce
Using Istio, we have a rewrite rule set up like this:

- match:
    - uri:
        prefix: /api/collector
    rewrite:
      uri: /api

So everything routed to https:///api/collector gets rerouted to https:///api

Expected behavior
We would like the API base path in https://github.com/razee-io/Razeedash-api/blob/1fe88efc21a2ba9e46aa8938fa9604828c0b7358/app/routes/install/index.js#L30 and everywhere else in the code to be configurable.

@nathanleviere nathanleviere added the enhancement New feature or request label Feb 12, 2020
@mckaymic
Copy link
Contributor

@thecheryl I'm going to put this in the backlog for now. This request can also be fixed with an ingress configuration.

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

No branches or pull requests

3 participants