search

web-platform-tests / rfcs

web-platform-tests RFCs
75 stars 63 forks source link

RFC 113: Remove Origin Policy infrastructure #113

Closed foolip closed 2 years ago

foolip commented 2 years ago

@domenic, as promised :)

gsnedders commented 2 years ago

While I don't have any objection to doing this, I do wonder if this is really worth the churn? There's risk here, and it's not clear there's much reward.

foolip commented 2 years ago

While I don't have any objection to doing this, I do wonder if this is really worth the churn? There's risk here, and it's not clear there's much reward.

Do you think the risk mitigation suggested is enough? On the reward side, since the output of ./wpt make-hosts-file is something most contributors will see while setting thing up, I think shrinking it from 260 to 62 lines is pretty nice, gives you less of a "WTF?" feeling and can be copied by hand into your hosts file if you don't want to just append it.

foolip commented 2 years ago

Discussed with @jgraham and @Ms2ger in the new RFCs/infra sync. We decided to go ahead with this, there's enough support and @gsnedders didn't quite object, but did wonder about the risk.

foolip commented 2 years ago

I've filed https://github.com/web-platform-tests/wpt/issues/34707 to implement this.