Skip to content

(PUP-6629) Be more careful about parsing YAML for transactionstore#5220

Merged
adrienthebo merged 1 commit intopuppetlabs:stablefrom
kbarber:ticket/stable/pup-6629-strip-yaml
Aug 19, 2016
Merged

(PUP-6629) Be more careful about parsing YAML for transactionstore#5220
adrienthebo merged 1 commit intopuppetlabs:stablefrom
kbarber:ticket/stable/pup-6629-strip-yaml

Conversation

@kbarber
Copy link
Contributor

@kbarber kbarber commented Aug 19, 2016

This patch strips Ruby tags from the transactionstore before parsing, so we avoid
some deserialization issues.

Signed-off-by: Ken Barber ken@bob.sh

@puppetcla
Copy link

CLA signed by all contributors.

@kbarber kbarber force-pushed the ticket/stable/pup-6629-strip-yaml branch 2 times, most recently from e97bf0c to fd24312 Compare August 19, 2016 19:10
This patch strips Ruby tags from the transactionstore before parsing, so we avoid
deserialization issues when classes inside types are problematic.

Signed-off-by: Ken Barber <ken@bob.sh>
@kbarber kbarber force-pushed the ticket/stable/pup-6629-strip-yaml branch from fd24312 to 345467c Compare August 19, 2016 19:13
@kbarber kbarber changed the title WIP (PUP-6629) Be more careful about parsing YAML for transactionstore (PUP-6629) Be more careful about parsing YAML for transactionstore Aug 19, 2016
@ferventcoder
Copy link
Contributor

Looks like it will do the needful

@adrienthebo adrienthebo merged commit 9f1eb72 into puppetlabs:stable Aug 19, 2016
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.

4 participants