Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

new-extension-version fails if manifest contains CRLF #9

Open
matt-richardson opened this issue Feb 9, 2017 · 1 comment
Open

new-extension-version fails if manifest contains CRLF #9

matt-richardson opened this issue Feb 9, 2017 · 1 comment

Comments

@matt-richardson
Copy link

I found, through lots of painful trial and error, that if your manifest file has CRLF line endings, it fails when pushing to azure.

It throws on the server side, and returns a 400 with no body.

Replacing CRLF with LF works fine.

(I suspect this is not the right repo for this issue, but not sure where to log. Maybe https://github.com/Azure/azure-marketplace is the right spot?)

@matt-richardson matt-richardson changed the title new-extension-version fails if file contains CRLF new-extension-version fails if manifest contains CRLF Feb 9, 2017
@boumenot
Copy link
Member

boumenot commented Feb 9, 2017

I don't think Azure/azure-marketplace is the correct repo. Let me see if I can connect this issue with the right people.

A couple of questions:

  1. Are you using this tool to deploy an extension?
    • If yes, we should be able to sanitize the input before sending it.
  2. Are you able to attach or reproduce a minimal repo?

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

No branches or pull requests

2 participants