Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Temporarily disable amp-script until interaction of amp-script and si… #23147

Merged
merged 2 commits into from
Jul 2, 2019
Merged

Temporarily disable amp-script until interaction of amp-script and si… #23147

merged 2 commits into from
Jul 2, 2019

Conversation

Gregable
Copy link
Member

@Gregable Gregable commented Jul 1, 2019

Temporarily disable amp-script until interaction of amp-script and signed exchanges has been security reviewed.

Greg Grothaus added 2 commits July 1, 2019 16:40
@Gregable
Copy link
Member Author

Gregable commented Jul 1, 2019

cc @choumx @kristoferbaxter

@Gregable Gregable merged commit b8e19c1 into ampproject:master Jul 2, 2019
@Gregable Gregable deleted the ampscript branch July 2, 2019 00:28
Gregable pushed a commit that referenced this pull request Jul 2, 2019
* cl/255021843 Add a test file illustrating the issue in #18091

* cl/255463519 Revision bump for #22679

* cl/256058522 Revision bump for #23043

* cl/256067928 Revision bump for #23135

* cl/256199406 Revision bump for #23147
thekorn pushed a commit to edelight/amphtml that referenced this pull request Sep 11, 2019
…roject#23147)

* Temporarily disable amp-script until interaction of amp-script and signed exchanges has
been security reviewed.

* Update tests.
thekorn pushed a commit to edelight/amphtml that referenced this pull request Sep 11, 2019
* cl/255021843 Add a test file illustrating the issue in ampproject#18091

* cl/255463519 Revision bump for ampproject#22679

* cl/256058522 Revision bump for ampproject#23043

* cl/256067928 Revision bump for ampproject#23135

* cl/256199406 Revision bump for ampproject#23147
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants