mweagle / Sparta

go microservices, powered by AWS Lambda
https://gosparta.io
MIT License
716 stars 48 forks source link

Fix PROTOCOL_ERROR on aws-go-sdk/apigateway/GetRestApi #44

Closed dhoeric closed 7 years ago

dhoeric commented 7 years ago

Got an error on go run main.go provision

Error message:

...
INFO[0022] Checking current APIGateway stage status      APIName=[REDACT]-staging StageName=staging
INFO[0022] DEBUG: Request apigateway/GetRestApis Details:
---[ REQUEST POST-SIGN ]-----------------------------
GET http://apigateway.ap-southeast-1.amazonaws.com/restapis?limit=500 HTTP/1.1
Host: apigateway.ap-southeast-1.amazonaws.com
User-Agent: aws-sdk-go/1.6.2 (go1.7.4; darwin; amd64)
Accept: application/json
Authorization: AWS4-HMAC-SHA256 Credential=[REDACT]/20170122/ap-southeast-1/apigateway/aws4_request, SignedHeaders=accept;host;x-amz-date, Signature=[REDACT]
X-Amz-Date: 20170122T080920Z
Accept-Encoding: gzip

-----------------------------------------------------
DEBU[0022] AWS Request                                   Method=GET Operation=GetRestApis Path="/restapis" Payload={
  Limit: 500
} Service=apigateway
INFO[0023] DEBUG: Response apigateway/GetRestApis Details:
---[ RESPONSE ]--------------------------------------
HTTP/0.0 000 status code 0
Content-Length: 0

-----------------------------------------------------
INFO[0023] DEBUG: Send Request apigateway/GetRestApis failed, will retry, error RequestError: send request failed
caused by: Get https://apigateway.ap-southeast-1.amazonaws.com/restapis?limit=500: stream error: stream ID 1; PROTOCOL_ERROR
...

Which has also mentioned on https://github.com/aws/aws-sdk-go/issues/1022. Welcome for any comments :)

mweagle commented 7 years ago

Thanks @dhoeric - I've updated all the dependencies and restored Travis CI functionality starting with https://travis-ci.org/mweagle/Sparta/builds/194240164. Appreciate the PR & closing in favor of https://github.com/mweagle/Sparta/commit/19050ee115d8cc92c9c2e22f52ac889b3240dae5 & related.

Can you confirm that resolves the issue & close this PR if it does?

dhoeric commented 7 years ago

Sure @mweagle , happy to see an active open source project 😄