Swap order of processing json_request to prevent bad jsonlines request.

aws / sagemaker-tensorflow-serving-container

A TensorFlow Serving solution for use in SageMaker. This repo is now deprecated.

Apache License 2.0

174 stars 101 forks source link

Swap order of processing json_request to prevent bad jsonlines request. #224

Closed matherit closed 1 year ago

matherit commented 1 year ago

Issue #, if available:

Description of changes: Swaps order on how json requests are processed. This should prevent valid TFS requests from being processed as if its a jsonlines type request.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

sagemaker-bot commented 1 year ago

AWS CodeBuild CI Report

CodeBuild project: sagemaker-tensorflow-serving-container-pr
Commit ID: 8a68e2c784947989d470149b9440db56bc5ee9fd
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

sagemaker-bot commented 1 year ago

AWS CodeBuild CI Report

CodeBuild project: sagemaker-tensorflow-serving-container-pr
Commit ID: 1615b2c0811c870a466df4480feb56cbc520bc55
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

sagemaker-bot commented 1 year ago

AWS CodeBuild CI Report

CodeBuild project: sagemaker-tensorflow-serving-container-pr
Commit ID: 2bf4e0f7b177dd78df27b29f614d87484f855c33
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository