replace double quotes with single quotes

YaleUniversity / packer-plugin-goss

Goss Provisioner for Packer

MIT License

136 stars 45 forks source link

replace double quotes with single quotes #49

Closed perithompson closed 3 years ago

perithompson commented 3 years ago

Found a bug on the windows command line that doesn't unmarshall inline variable json strings into goss that have " and not single quotes e.g. Not working {"OS":"windows"} working {'OS':'windows'}

perithompson commented 3 years ago

@jsturtevant do you see the same? this seems to resolve it for me but I'm not sure if this should be fixed in Goss itself, but for now this works around the problem

perithompson commented 3 years ago

we can also get around this by escaping the " like {\"OS\":\"windows\"} but I found that more messy to read...

perithompson commented 3 years ago

Hey @fishnix, are you able to review and let me know if this looks ok to merge?