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

Should Run Before SRI Integrity Generation #96

Closed
wants to merge 1 commit into from

Conversation

kalebdf
Copy link

@kalebdf kalebdf commented Apr 25, 2016

If this plugin runs CSS & JS generation after ember-cli-sri has run, the running app will fail to find a valid digest in the integrity attribute for the resource.

It seems to be similar to this: ember-cli/ember-cli#5040 and this issue in ember-paper

It seems to be an issue in Chrome only. No problem when using Firefox.

If this runs after the CSS & JS generation and the ember-cli-sri has run, the running app will fail to find a valid digest in the `integrity` attribute for the resource.
@aaronbhansen
Copy link
Contributor

Thanks for the pull request. I've read through the comments on the referenced, but couldn't see a clear answer as to why it was failing for some and not for others. I've personally never run into this issue on our projects. The change is small, but i want to ensure it doesn't effect other people by merging that currently don't have the issue. Was it just a specific version of ember or sri that you had this error?

@kalebdf
Copy link
Author

kalebdf commented May 27, 2016

@aaronbhansen The versions are Ember 2.4.5 and Ember-Data 2.5.3 and ember-cli-sri 2.1.0. It's a weird ordering issue indeed.

@aaronbhansen
Copy link
Contributor

@kalebdf is this still an issue for you? I haven't run into or had anyone else with this issue. Looking to close this out if its resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants