Skip to content

Conversation

@jeoy
Copy link
Contributor

@jeoy jeoy commented Feb 26, 2018

according https://github.com/Microsoft/code-push/blob/master/sdk/script/management-sdk.ts#L337

   if (expectResponseBody && !body) {
        reject({ message: "Could not parse response: " + res.text, statusCode: res.status });
   }

if code-push successfully release a new packages with sending '' as response text, we always encounter this error 'Could not parse response:'.

Instead we should send a JSON string as response text, like '{"msg": "succeed"}'.

@lisong lisong merged commit 020a350 into lisong:master Mar 2, 2018
plingdevelop pushed a commit to plingdevelop/code-push-server that referenced this pull request Dec 22, 2025
fix(deps): update dependency sequelize to v6.16.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants