#141 change delayed_job.handler to mediumtext (16MB) like event.payload

[Alex-Ikanow]

This isn't a complete fix because if the event.payload is (say) exactly
16MB then the handler will be >16MB and the same problem will occur.

I'll push this in for now though, in practice it means it's much less
likely to occur.

[cantino]

Thanks!

[Alex-Ikanow]

Should you find yourself anywhere near the payload->handler code at some point, you might want to check the ==16MB payload code - does the handler get corrupted by the preamble/postamble? (If so it's probably as simple as actually limiting the payload to 16MB - delta). Do you want me to keep the parent issue open so one of us can address that (probably small) hole at some point, close it, or create another?

[cantino]

That's a good point. You could make a new issue for it.

On Fri, Jan 31, 2014 at 1:32 PM, Alex notifications@github.com wrote:

Should you find yourself anywhere near the payload->handler code at some
point, you might want to check the ==16MB payload code - does the handler
get corrupted by the preamble/postamble? (If so it's probably as simple as
actually limiting the payload to 16MB - delta). Do you want me to keep the
parent issue open so one of us can address that (probably small) hole at
some point, close it, or create another?

Reply to this email directly or view it on GitHubhttps://github.com//pull/152#issuecomment-33844555
.