The new param ampCors: boolean introduced by #6637 should replace the existing requireAmpResponseSourceOrigin.
No new code should specify requireAmpResponseSourceOrigin. Rather, they should use ampCors: false to turn off AMP CORS, which means no __amp_source_origin in XHR URL, and no check for AMP_SOURCE_ORIGIN response header.
We will make ampCors defaults to true. (Or false? @cramforce thoughts? Do we encourage people to enable AMP CORS?).
For backward compatibility, will change the existing code to include requireAmpResponseSourceOrigin: false explicitly if they still want the __amp_source_origin in request, but no check for response.
The new param
ampCors: booleanintroduced by #6637 should replace the existingrequireAmpResponseSourceOrigin.No new code should specify
requireAmpResponseSourceOrigin. Rather, they should useampCors: falseto turn off AMP CORS, which means no__amp_source_originin XHR URL, and no check for AMP_SOURCE_ORIGIN response header.We will make
ampCorsdefaults totrue. (Or false? @cramforce thoughts? Do we encourage people to enable AMP CORS?).For backward compatibility, will change the existing code to include
requireAmpResponseSourceOrigin: falseexplicitly if they still want the__amp_source_originin request, but no check for response.