Skip to content

Test Updates to Align with Header Removal from APIcast Midstream - #1029

Open
cathal-bailey wants to merge 1 commit into
3scale-qe:mainfrom
cathal-bailey:onfailed-openresty-headers-removed
Open

Test Updates to Align with Header Removal from APIcast Midstream#1029
cathal-bailey wants to merge 1 commit into
3scale-qe:mainfrom
cathal-bailey:onfailed-openresty-headers-removed

Conversation

@cathal-bailey

@cathal-bailey cathal-bailey commented May 1, 2026

Copy link
Copy Markdown
Collaborator
  • APIcast midstream update removes the Server header from HTTP responses. This breaks tests that rely on checking headers to confirm APIcast is responding.
  • Update to the backoff mechanism to look for valid response code rather than server header. Ensures backoff logic remains in place and working, in case it is needed.
  • test_do_not_send_openresty_version.py‎ also affected by midstream change.
  • Test now verifies that no server header or version information is disclosed anywhere.

@cathal-bailey
cathal-bailey force-pushed the onfailed-openresty-headers-removed branch from 9a1e120 to 33fc4ad Compare May 1, 2026 13:27
@cathal-bailey
cathal-bailey marked this pull request as ready for review May 1, 2026 13:28
mdujava
mdujava previously approved these changes May 4, 2026
@cathal-bailey cathal-bailey changed the title Removing Headers From onfailed Policy Tests Test Updates to Align with Header Removal from APIcast Midstream May 8, 2026
@mdujava

mdujava commented May 11, 2026

Copy link
Copy Markdown
Member

are there any upstream changes or jira issue that shows this is expected change?

@cathal-bailey

cathal-bailey commented May 11, 2026

Copy link
Copy Markdown
Collaborator Author

are there any upstream changes or jira issue that shows this is expected change?

@mdujava

mdujava commented May 11, 2026

Copy link
Copy Markdown
Member

Comment thread testsuite/tests/apicast/policy/on_failed/test_onfailed.py Outdated
@cathal-bailey
cathal-bailey force-pushed the onfailed-openresty-headers-removed branch from 82557c9 to d0efaaa Compare May 18, 2026 09:38
mastastny
mastastny previously approved these changes May 18, 2026
@cathal-bailey
cathal-bailey requested a review from mdujava August 11, 2026 14:28
Comment thread testsuite/tests/apicast/test_do_not_send_openresty_version.py Outdated
Comment thread testsuite/tests/apicast/test_do_not_send_openresty_version.py Outdated
@cathal-bailey
cathal-bailey force-pushed the onfailed-openresty-headers-removed branch from 66ea169 to 52f2997 Compare August 12, 2026 10:19
@cathal-bailey
cathal-bailey requested a review from mdujava August 12, 2026 10:20
@cathal-bailey
cathal-bailey force-pushed the onfailed-openresty-headers-removed branch 2 times, most recently from 2bc3d3d to 522b354 Compare August 14, 2026 08:15
Add ZYNC and SSO capabilities with ZyncLessApicast support

Introduce ZYNC_ROUTES and ZYNC_OIDC_SYNC capabilities that reflect the
actual zync deployment state (enabled, routes disabled, or fully disabled).
Add ZyncLessApicast gateway variant that manually manages OCP routes via
lifecycle hooks when zync is disabled. Add fail-fast validation at session
start to catch zync config mismatches early. Annotate affected tests with
the new capabilities.

Wait for APIcast 503 response

Add ZYNC and SSO capabilities with ZyncLessApicast support

Introduce ZYNC_ROUTES and ZYNC_OIDC_SYNC capabilities that reflect the
actual zync deployment state (enabled, routes disabled, or fully disabled).
Add ZyncLessApicast gateway variant that manually manages OCP routes via
lifecycle hooks when zync is disabled. Add fail-fast validation at session
start to catch zync config mismatches early. Annotate affected tests with
the new capabilities.

Fix formatting issues - add blank lines and remove trailing whitespace
@cathal-bailey
cathal-bailey force-pushed the onfailed-openresty-headers-removed branch from 67a6c31 to c48a0a3 Compare August 18, 2026 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants