Azure / MLAKSDeployment

How to deploy Python models on a Kubernetes cluster
MIT License
14 stars 11 forks source link

Adding Application Insights and API Key security #6

Closed grecoe closed 6 years ago

grecoe commented 6 years ago

Add to the creation of the images two new files - applicationinsights.py and apisecurity.py

applicationinsights.py: Commented out but shows how to use Azure Application Insights in the flask app. apisecurity.py: Provides a basic Bearer token authorization for the score endpoint

Score endpoints updated to require "Authorization" : "Bearer KEY" and all test notebooks updated to use the new auth header with the key provided in apisecurity.py

msalvaris commented 6 years ago

Have a look at conversation here We are trying to figure out best way to have these additions to the paths.

grecoe commented 6 years ago

Conflicting opinions on this, one of the repo owners can close this request and delete the branch.

marabout2015 commented 6 years ago

There are questions about whether the feature should be added to this path.