Should relative hosts support be removed from optimizer?

[mdmower]

Based on https://github.com/ampproject/amphtml/issues/27546 , it sounds like absolute, HTTPS URLs should be required for parameter ampUrlPrefix used in DomTransformer.transformHtml().

In optimizer,

• should we remove support for relative hosts?
• should we require HTTPS URLs?

We should keep in mind that some users may want to test optimized AMP pages in local dev environments (e.g. localhost or HTTP), so it may be best to make no changes.

[mdmower]

My recommendation after looking at optimizer for a bit: require absolute URLs. This is mostly so that AmpBoilerplateTransformer can inline the correct v0.css. I can go either way on allowing or forbidding HTTP URLs.

Work in progress: https://github.com/mdmower/amp-toolbox/commits/pr-absolutehost

[sebastianbenz]

+1 to requiring absolute hosts