Skip to content

Conversation

@lukeed
Copy link
Member

@lukeed lukeed commented Feb 11, 2018

What kind of change does this PR introduce?

bugfix

Did you add tests for your changes?

No

Summary

You can accidentally include .git* files inside your sw-precache assets, especially with a non-standard directory setup. For example, having src/assets/.gitkeep will try to cache .gitkeep at runtime, resulting in this error.

Uncaught (in promise) Error: Request for https://.../assets/icons/.gitkeep?_sw-precache=d41d8cd98f00b204e9800998ecf8427e returned a response with status 404

@reznord reznord merged commit 0feddca into master Feb 11, 2018
@lukeed lukeed deleted the fix/sw-ignore branch February 11, 2018 20:48
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.

4 participants