chore(package): update webpack-log v1.0.1...1.2.0 (dependencies)

[svicalifornia]

Fixes #317.

[jsf-clabot]

All committers have signed the CLA.

[svicalifornia]

So updating a dependency to the version mutually recommended in #310 breaks your test suite. Well, that's fun. I'll have to come back to fix this later, unless someone else can help sooner.

[alexander-akait]

@svicalifornia please fix tests

[svicalifornia]

@evilebottnawi I already said above that I would. It's gonna take me a few days to get back to this, though.

[michael-ciniawsky]

The test failures are weird tbh, it's likely something changed in webpack itself making the bundle a bit larger. Also why is it required to be exactly 3645 in size ? I would suggest to either update the expected value(s) to the currently recieved ones or to remove this brittle assertion altogether. Anyways it seems this is unrelated to changes made by the PR itself

[alexander-akait]

@michael-ciniawsky can we fix it in this PR?

[michael-ciniawsky]

I think so, but I'm not 💯 sure if the content-length check has an important meaning or not, from the test suit it looks like it doesn't (app.js is e.g not an injected script by the middleware or the like, just a test app.js bundle (which can change in size easily)). I would suggest to update the size to the currently recieved value for now and add a comment that this needs triage and refactor from our side :)

[alexander-akait]

@michael-ciniawsky let's do it, feel free create separate PR for this

[alexander-akait]

/cc @michael-ciniawsky need patch release, can you have access for this?

[michael-ciniawsky]

No 😞 and I just saw you also don't have 🤢...

[alexander-akait]

@michael-ciniawsky waiting for access