Remove pagespeed module from our nginx container

[jometzner]

PR Type

[ ] Bugfix
[ X ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] CI-related changes
[ ] Documentation content changes
[ ] Application / infrastructure changes
[ ] Other:

What Is the Current Behavior?

Currently we use all sorts of filters included in the pagespeed module of NGINX. The contribution to actual page speed ranking is not 100% clear. Especially since the introduction of core web vitals it would be beneficial to re-evaluate the usage of this module.

Con List:

1. Inactive community
2. Old release
3. Performance impact to core web vitals unclear
4. Increases the burden on SSR process with periodic beacon calls
5. Longer docker builds

What Is the New Behavior?

Does this PR Introduce a Breaking Change?

[ X ] Yes
[ ] No

Other Information

AB#76451

[dhhyi]

@jometzner That was one of the first major deployment improvements when we initially set up the demo servers. And at the beginning it also yielded a highscore in performance. Of course a lot has changes since then and due to the added PWA complexity (and maintenance status recently shown by the pinning of a 2yo ubuntu base image) it might not be useful anymore.

I like the refactoring of using built-in nginx startup scripts. 👍

This might be the way to go for a maintainable PWA, still, I'm sad to see it go away 😭

[jometzner]

Well the module is not gone yet. We're re-evaluating the decision from the first days to see if it still fits the requirements. I can still remember that it did good on the page speed results.

[shauke]

First local tests did not show any changes in the lighthouse performance results.
And it now works on M1 Macs. 👍
And it is mach faster too. So I am totally for it. 😉

[jometzner]

You could give the pull request a thumbs up

[jometzner]

I'll write some documentation for this change before it gets merged

[dhhyi]

Haven't tested in our project yet, will do that next.

[jometzner]

I'll write some documentation for this change before it gets merged

✅