Skip to content

Conversation

@n7studios
Copy link
Contributor

@n7studios n7studios commented Dec 3, 2025

Summary

Tests against Uncode's theme started returning a 502 bad gateway in nginx, when run by GitHub actions (locally, these tests pass):
Tests EndToEnd RestrictContentProductThirdPartyThemeOrPageBuilderCest testRestrictContentByProductWithUncodeThemeAndVisualComposer fail

Configuring the GitHub action to upload /var/log/nginx/error.log as an artifact on failure revealed:
2025/12/04 04:47:02 [error] 2700#2700: *45 upstream sent too big header while reading response header from upstream, client: 127.0.0.1, server: localhost, request: "POST /post-2-title/?convertkit_login=1 HTTP/1.1", upstream: "fastcgi://unix:/run/php/php8.2-fpm.sock:", host: "127.0.0.1", referrer: "http://127.0.0.1/post-2-title/?ck-cache-bust=0.11952200%201764823621"

This PR resolves by increasing the buffer size in nginx, and includes uploading nginx's error.log to the GitHub action artifacts, should tests in the future fail.

Checklist

@n7studios n7studios self-assigned this Dec 3, 2025
@n7studios n7studios added the bug label Dec 3, 2025
@github-actions
Copy link

github-actions bot commented Dec 4, 2025

WordPress Playground

🚀 Your PR has been built and is ready for testing in WordPress Playground!

Click here to test your changes in WordPress Playground

@github-actions
Copy link

github-actions bot commented Dec 4, 2025

WordPress Playground

🚀 Your PR has been built and is ready for testing in WordPress Playground!

Click here to test your changes in WordPress Playground

@github-actions
Copy link

github-actions bot commented Dec 4, 2025

WordPress Playground

🚀 Your PR has been built and is ready for testing in WordPress Playground!

Click here to test your changes in WordPress Playground

@n7studios n7studios requested review from a team, ciccio-kit and noelherrick and removed request for a team December 4, 2025 09:50
@n7studios n7studios marked this pull request as ready for review December 4, 2025 09:50
@n7studios n7studios merged commit 9b22e1b into main Dec 5, 2025
217 of 227 checks passed
@n7studios n7studios deleted the tests-fix-uncode-502 branch December 5, 2025 06:53
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