[midPoint-git] [Evolveum/connector-scimrest] 41b2e0: Fix misleading JDoc on RestContext.newAuthorizedRe...

[johnstone3]

  Branch: refs/heads/fix/scim-http-error-and-test-auth-2
  Home:   https://github.com/Evolveum/connector-scimrest
  Commit: 41b2e087e34f88a85788fdeb335a524b2f382d7b
      https://github.com/Evolveum/connector-scimrest/commit/41b2e087e34f88a85788fdeb335a524b2f382d7b
  Author: Jan Kamenicky <jan.kamenicky at evolveum.com>
  Date:   2026-05-25 (Mon, 25 May 2026)

  Changed paths:
    M common/src/main/java/com/evolveum/polygon/scimrest/impl/rest/RestContext.java

  Log Message:
  -----------
  Fix misleading JDoc on RestContext.newAuthorizedRequest()

The method only initializes the request with base URI and timeout;
authorization is applied later in executeRequest().


  Commit: 20254dd6bb11e091c687769c19b847128b47d2ab
      https://github.com/Evolveum/connector-scimrest/commit/20254dd6bb11e091c687769c19b847128b47d2ab
  Author: Jan Kamenicky <jan.kamenicky at evolveum.com>
  Date:   2026-05-25 (Mon, 25 May 2026)

  Changed paths:
    M common/src/main/java/com/evolveum/polygon/scimrest/groovy/AbstractGroovyRestConnector.java
    M common/src/main/java/com/evolveum/polygon/scimrest/impl/scim/ScimContext.java
    A common/src/main/java/com/evolveum/polygon/scimrest/impl/scim/ScimHttpErrorFilter.java
    M common/src/test/java/com/evolveum/polygon/scimrest/exception/ScimExceptionSimulationTest.java
    M common/src/test/java/com/evolveum/polygon/scimrest/exception/auth/basic/BasicAuthTests.java
    M common/src/test/java/com/evolveum/polygon/scimrest/exception/auth/basic/ScimHttpBasicTests.java
    M common/src/test/java/com/evolveum/polygon/scimrest/exception/auth/bearertoken/ScimBearerTokenAuthTests.java
    M common/src/test/java/com/evolveum/polygon/scimrest/exception/auth/oauth2clientcredentials/ScimOAuth2ClientCredentialsTests.java

  Log Message:
  -----------
  Fix SCIM connector: send auth headers to REST test endpoint and handle HTTP errors cleanly

ScimHttpErrorFilter intercepts 4xx/5xx responses before the SCIM2
library attempts JSON-B deserialization of ErrorResponse, avoiding
JsonbException in the cause chain.

ScimContext exposes the Jersey HTTP client (httpClient()); AbstractGroovyRestConnector.test()
uses this client when SCIM is active so the JerseyRequestCustomizerFilter
(which holds all registered auth customizers) applies to the test
endpoint request — fixing Bearer, Basic, and OAuth2 auth not being sent.

Tests added for each auth type verifying auth headers reach the test endpoint
in both REST (BasicAuthTests) and SCIM (ScimBearerTokenAuthTests,
ScimHttpBasicTests, ScimOAuth2ClientCredentialsTests) files.


Compare: https://github.com/Evolveum/connector-scimrest/compare/41b2e087e34f%5E...20254dd6bb11

To unsubscribe from these emails, change your notification settings at https://github.com/Evolveum/connector-scimrest/settings/notifications