proposed postMessage rev

proposed postMessage rev

[Kevin Carhart]

I think we chewed on a couple of changes to postMessage but they aren't in yet.  What do you think of the following?

The differences are
(1) we are supposed to consider port (as in, networks, 80, 443, not talking about MessageChannel stuff here) when passing or failing the origin test
(2) setting me.origin = my$win().location;
(3) Adding an else case to the origin test so that if the postMessage can't run, we can be alerted that it didn't

-------------



function  postMessage(message,target_origin, transfer) {
var locstring = this.location.protocol + "//" + this.location.hostname + ":" + this.location.port;
if (target_origin == locstring || target_origin == "*") {
var me = new Event;
me.name = "message";
me.type = "message";
me.origin = my$win().location;
me.data = message;
me.source = my$win();
if(transfer) {
me.ports = transfer;
// If these objects had a context, they are now owned by this context.
for(var i = 0; i < transfer.length; ++i)
if(transfer[i].eb$ctx) transfer[i].eb$ctx = eb$ctx;
}
this.onmessage$$queue.push(me);
alert3("posting message of length " + message.length + " to window context " + this.eb$ctx + " ?" +
(message.length >= 200 ? "long" : message)
+ "?");
} else {
// handle a failure here
alert3("postMessage failure: the message " + message + " could not be posted to " + this.eb$ctx + " due to the same-origin rule");
}
}

Re: proposed postMessage rev

[Adam Thompson]

On Wed, Dec 22, 2021 at 07:37:53PM -0800, Kevin Carhart wrote:
> I think we chewed on a couple of changes to postMessage but they aren't in yet.  What do you think of the following?
> 
> The differences are
> (1) we are supposed to consider port (as in, networks, 80, 443, not talking about MessageChannel stuff here) when passing or failing the origin test
> (2) setting me.origin = my$win().location;
> (3) Adding an else case to the origin test so that if the postMessage can't run, we can be alerted that it didn't

If it's not already added, it all looks sensible to me.