trotzig
commented
7 months ago Hi @marcellmueller! Sorry for the turbulence with the Safari workers. I made some additional changes on Thursday to mitigate this when I found that one of our worker/server machines had a slightly different spec than the others. I ran through some tests and everything looked stable. But when I'm looking into your issue now I realize that we have a stale cache issue. I'll try to explain here, but feel free to ignore if it's TMI. 🙂
Under certain circumstances, Happo can use cached screenshots. The following criteria have to be met:
- The pre-rendered HTML/markup is exactly the same as before
- The CSS is exactly the same
- The assets are exactly the same
- There are no external requests
- The worker that produced the screenshot is running the same Happo worker version
- Plus some other stuff that I forget 😊
Pre-rendered HTML is used when using Cypress, Playwright or Happo Examples integration with prerender: true.
In the case of the recent Safari upgrade, we didn't bump the Happo worker version. This meant that caches produced by old workers were in effect.
I'm addressing the caching issue right away. Keep me updated here if you continue to have issues even after merging a new PR to create a new baseline.
marcellmueller
Recently there was a warning message about a Safari browser update that might cause some diffs though we are still getting them after this with multiple PRs merged to update the baseline. Is this going to go away or should we increase the diff threshold? Thanks for all your hard work on Happo by the way, our team loves using it 🙇