Serializing Objects with Functions causes an error

[aaronius]

Serializing Objects with Functions causes an error

[Aaronius]

Thanks for logging the issue @OverZealous. I think I prefer the current behavior of an error being thrown, so that it's more obvious to the developer that they can't send a function to/from the iframe. I do think I need to be more explicit about this nuance in the Penpal documentation though.

FWIW, I think you can change your function to this:

export function toSafeObject(data) {
  return JSON.parse(JSON.stringify(data));
}

The native JSON serialization process automatically strips functions. I'm going to close this issue, but feel free to continue to comment or add questions.

[OverZealous]

I understand. I wish the browser-generated console error was better—it's not very clear what's triggering it.

One reason it would be nice to automatically handle this is you don't have to wrap methods or results from methods that might be shared within the parent or child application. This is useful for triggering actions that can be called on either side by simply sharing the same method.

I did look at the stringify method before, but I believe it strips out values that are valid to send across postMessage. I think you can serialize several data types that aren't supported within JSON, such as NaN, Infinity, Symbol, Errors, and the various *Array objects like Int8Array. Though admittedly, my method isn't that robust against the arrays and friends.

I agree It's not critical enough to add to the library. However, would it make sense to add serialize & deserialize options to the library? This would allow the user to clean up any data in a concise manner.

[Aaronius]

Thanks for pointing that out. Let me keep this in mind. If it comes up again, it might be worth adding serialize and deserialize options. Thanks again.