-
Notifications
You must be signed in to change notification settings - Fork 4
Conversation
doc/api-doc.yml
Outdated
@@ -1,13 +1,12 @@ | |||
info: | |||
title: A Qlik Associative Engine discovery service | |||
version: 0.1.2 | |||
version: 0.2.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this should be a change of API version since we change the paths?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this be 0.2.0
@@ -26,15 +25,15 @@ paths: | |||
get: | |||
description: Returns metrics of the Mira service | |||
produces: | |||
- application/json; charset=utf-8 | |||
- application/json |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed to only application/json since that is what our metrics library gives out.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also JSON according to the spec must be utf-8 so it should be implicit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
This closes #274 and #275
This PR removes /v1/metrics and /v1/health