Skip to content

Remove fingerprint middleware#20682

Merged
pvizeli merged 2 commits into
devfrom
remove-fingerprint
Feb 2, 2019
Merged

Remove fingerprint middleware#20682
pvizeli merged 2 commits into
devfrom
remove-fingerprint

Conversation

@balloob
Copy link
Copy Markdown
Member

@balloob balloob commented Feb 2, 2019

Description:

We used to be able to add a 32 char hash to each filename to bust cache. This was dangerous and often ended up with files not being served correctly (older firmware etc, messing up with service workers).

After home-assistant/frontend#2652 is merged and the frontend is updated in the backend, we will no longer need it.

@balloob balloob requested a review from a team as a code owner February 2, 2019 06:14
@ghost ghost assigned balloob Feb 2, 2019
@ghost ghost added the in progress label Feb 2, 2019
@pvizeli pvizeli merged commit 47f60e6 into dev Feb 2, 2019
@ghost ghost removed the in progress label Feb 2, 2019
@delete-merged-branch delete-merged-branch Bot deleted the remove-fingerprint branch February 2, 2019 10:52
balloob added a commit that referenced this pull request Feb 3, 2019
* Remove fingerprint middleware

* Lint
@balloob balloob mentioned this pull request Feb 6, 2019
@balloob balloob mentioned this pull request Feb 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants