Skip to content

Should relative hosts support be removed from optimizer? #697

@mdmower

Description

@mdmower

Based on ampproject/amphtml#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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions