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

Already on GitHub? Sign in to your account

Try to load `payload_object` created with `YAML::syck` parser #641

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants

This pull-request aims to fix a rare case when handler contains YAML-encoded data that was serialized with syck YAML parser. delayed_job crashes when worker tries to deserialize this data.

Although this issue might be very specific and subjective to our application I still would like to submit this fix. Let me know if this can be merged and/or I should add some tests for this case.

Thanks,
Igor

I ran into this issue this morning.

Owner

albus522 commented Sep 19, 2014

This would not load the syck extensions DJ needs and is a rare code path we would not want to maintain going forward.

@albus522 albus522 closed this Sep 19, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment