ampproject / amphtml

The AMP web component framework.
https://amp.dev
Apache License 2.0
14.89k stars 3.89k forks source link

Incapsula CDN adds analytics javascript to pages, breaks validation #5051

Closed weeblr closed 7 years ago

weeblr commented 8 years ago

As per @cramforce request, opening a new issue: one of our users reported validation issue on AMP pages. Turns out they are using Imperva Incapsula CDN, which adds a piece of javascript for tracking to all pages: https://monosnap.com/file/YEGachNyT5bYelENvdJa3HgFrzyvdu

Should be noted that:

Most users have no clue why their pages do not vallidate, and when informed, some cannot disable caching per URL because they are on free accounts, so they must disable CDN entirely. That's an obstacle to adoption I think.

As a follow up to previous issue #2380 , maybe it would be good to contact CDN providers and inform them about the necessity to leave AMP pages alone and not inject stuff in them, like Cloudflare did?

Rgds

weeblr commented 8 years ago

FYI, Incapsuala replied to a question on tweeter with :

can be fixed upon request, working on automatic solution.

They should post here when automatic solution is active.

Incaprules commented 8 years ago

Thanks for opening this issue, I'm from Incapsula, will update here once we have a resolution

thisizkp commented 8 years ago

@Incaprules @rudygalfi what is the status of the issue ? any resolution so far ?

ZigZagIL commented 7 years ago

The solution was evaluated by a few customers and is now being deployed globally.

adelinamart commented 7 years ago

What's the latest update on this? Thanks

Incaprules commented 7 years ago

Incapsula supports AMP pages for all websites since Nov 2016

weeblr commented 7 years ago

I assume this can be closed. So I am closing it.