Add TO Go caches/stats endpoint #2278
Conversation
rob05c
added
new feature
|
Refer to this link for build results (access rights to CI server needed): |
rob05c
force-pushed
the
to-go-caches-stats
branch
from
3092455
to
1d33436
Compare
|
Refer to this link for build results (access rights to CI server needed): |
rob05c
force-pushed
the
to-go-caches-stats
branch
from
1d33436
to
d2d3309
Compare
|
Refer to this link for build results (access rights to CI server needed): |
rob05c
force-pushed
the
to-go-caches-stats
branch
from
d2d3309
to
1a292ce
Compare
|
Refer to this link for build results (access rights to CI server needed): |
rob05c
force-pushed
the
to-go-caches-stats
branch
from
1a292ce
to
fe64f64
Compare
|
Refer to this link for build results (access rights to CI server needed): |
rob05c
force-pushed
the
to-go-caches-stats
branch
2 times, most recently
from
b840ce8
to
864a1d1
Compare
|
Refer to this link for build results (access rights to CI server needed): |
rob05c
force-pushed
the
to-go-caches-stats
branch
2 times, most recently
from
067edcc
to
22df4a4
Compare
|
Refer to this link for build results (access rights to CI server needed): |
|
Refer to this link for build results (access rights to CI server needed): |
Previously, the endpoint returned an error if the first monitor it tried failed. This changes it to retry all online monitors.
rob05c
force-pushed
the
to-go-caches-stats
branch
from
22df4a4
to
544906c
Compare
|
Refer to this link for build results (access rights to CI server needed): |
|
@ocket8888 is going to review this PR |
| @@ -103,6 +104,8 @@ func Routes(d ServerData) ([]Route, []RawRoute, http.Handler, error) { | |||
| {1.1, http.MethodPost, `asns/?$`, api.CreateHandler(asn.GetTypeSingleton()), auth.PrivLevelOperations, Authenticated, nil}, | |||
| {1.1, http.MethodDelete, `asns/{id}$`, api.DeleteHandler(asn.GetTypeSingleton()), auth.PrivLevelOperations, Authenticated, nil}, | |||
|
|
|||
| {1.1, http.MethodGet, `caches/stats/?(\.json)?$`, cachesstats.Get, auth.PrivLevelReadOnly, Authenticated, nil}, | |||
There was a problem hiding this comment.
I don't think this is unique to this endpoint, but this will allow me to GET /api/1.x/caches/stats/.json. I checked and it seems that the Perl allows this, so maybe for that reason alone we don't want to change it, but it's weird.
This endpoint queries the Traffic Monitor; the "API Test" framework doesn't currently support endpoints that hit the Monitor.