kytos-ng / telemetry_int

Kytos Telemetry Napp
MIT License
0 stars 2 forks source link

chore: refactored `flask` imports with `starlette` related imports #1

Closed viniarck closed 1 year ago

viniarck commented 1 year ago

@jab1982, when you update kytos you'll need this PR just so it works with starlette that has landed upstream, if you haven't updated kytos and the other NApps locally yet then you can still use as it is though (and then once we move this NApp to kytos-ng org we can create a task there).

Summary

Local Tests

❯ echo '{"evc_ids": []}' | http http://localhost:8181/api/amlight/telemetry/v1/evc/enable
HTTP/1.1 200 OK
content-length: 46
content-type: application/json
date: Wed, 10 May 2023 16:55:10 GMT
server: uvicorn

{
    "f36361422f3149": "no proxy ports available."
}

End-to-End Tests

N/A

viniarck commented 1 year ago

I'll go ahead and merge this since it's pre-requisite to unblock the other tasks. I'll deal with git conflicts from https://github.com/kytos-ng/telemetry/issues/12 later.