8000 ExternalLinksTest is broken · Issue #48788 · keycloak/keycloak · GitHub
[go: up one dir, main page]

Skip to content

ExternalLinksTest is broken #48788

Closed as duplicate
Closed as duplicate

Description

@mabartos

Before reporting an issue

  • I have read and understood the above terms for submitting issues, and I understand that my issue may be closed without action if I do not follow them.

Area

docs

Describe the bug

When the documentation CI job is executed, the ExternalLinksTest is very unstable/broken:

Error:  Tests run: 5, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 42.10 s <<< FAILURE! -- in org.keycloak.documentation.test.ExternalLinksTest
Error:  org.keycloak.documentation.test.ExternalLinksTest.checkExternalLinks(String)[3] -- Time elapsed: 18.25 s <<< FAILURE!
org.opentest4j.AssertionFailedError: 
Broken links (1) in guide 'server_admin': 
		- https://www.oasis-open.org/standard/saml/ (invalid status code 403)
	at org.junit.jupiter.api.AssertionUtils.fail(AssertionUtils.java:42)
	at org.junit.jupiter.api.Assertions.fail(Assertions.java:143)
	at org.keycloak.documentation.test.ExternalLinksTest.checkExternalLinks(ExternalLinksTest.java:41)

I've also noticed it in some other PRs for other links, so maybe they have some kind of additional protection for non-human interaction? Meaning that the HTTP client cannot access it?

Or it's some kind of unintented behavior on their side?

Version

main

Regression

  • The issue is a regression

Expected behavior

The test pass

Actual behavior

The test fails

How to Reproduce?

Execute the test

Anything else?

We might try to retry requests even for 4xx HTTP response codes. EDIT: Not helped. Ignoring the website.

Activity

  1. self-assigned this
    on May 7, 2026
  2. added theissue type on May 7, 2026
  3. added
    area/docsDocumentation — server guides, AsciiDoc, JavaDoc, migration/upgrade guides
    on May 7, 2026
  4. changed the title [-]ExternalLinksTest is unstable[/-] [+]ExternalLinksTest is broken[/+] on May 7, 2026
  5. rmartinc commented on May 7, 2026

    @rmartinc
    Contributor

    It's a duplicate @mabartos. The PR is already sent is already merged. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

area/docsDocumentation — server guides, AsciiDoc, JavaDoc, migration/upgrade guideskind/bugCategorizes a PR related to a bugpriority/importantMust be worked on very soon

Type

Fields

No fields configured for bug.

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

    0