Add quarkus-jackson to quarkus-oidc

[sberyozkin]

Fixes #5342.

[sberyozkin]

Hi @gsmet @cescoffier I'm afraid of breaking things and not adding a Backport label, I'd rather say to 1.0.0 users they need to add a quarkus-jackson dependency manually if they need to run the oidc adapter in the native mode

[gsmet]

Failure is unrelated, merging.

[gsmet]

@sberyozkin what's the relation between adding quarkus-jackson and removing the Keycloak test resource?

Blocking the merge until I've got an answer for that.

[sberyozkin]

@gsmet not related, but I was cleaning up along the way, while checking the tests, looks like they were needed at some point of time to ensure keycloak.url was available to the tests (cc @pedroigor ) but I've checked if I can remove them and it worked :-)

[gsmet]

And it works locally too without a Keycloak server started? Because I seem to recall we have one running on CI.

[pedroigor]

@gsmet not related, but I was cleaning up along the way, while checking the tests, looks like they were needed at some point of time to ensure keycloak.url was available to the tests (cc @pedroigor ) but I've checked if I can remove them and it worked :-)

I've added that due to

quarkus/integration-tests/oidc/src/test/java/io/quarkus/it/keycloak/KeycloakTestResource.java

Line 13 in c497d79

// a workaround to set system properties defined when executing tests. Looks like this commit introduced an

. If it works now without it, +1 to remove it.

Are you able to change the url when running locally?

[sberyozkin]

@pedroigor @gsmet do you mean if I've run the tests locally ? Yes I did. May be worth double-checking, Pedro, please try ?

[sberyozkin]

@pedroigor I was not changing the URLs with -Dkeycloak.utl=someother url- I just run the tests with -Pdocker-keycloak -Ptest-keycloak. I can restore those KeycloakResource files if that is still needed in some situations

[sberyozkin]

@gsmet by the way I'm not sure if the OIDC native test passed in CI, looks like the job was terminated due to to some download issue ? Sorry, see it now :-) at https://dev.azure.com/quarkus-ci/quarkus/_build/results?buildId=12180

[sberyozkin]

@gsmet IMHO this is good to go, as I said I've confirmed the native tests work in the master, both oidc and oidc-code-flow against a locally launched KC image. But the actual fix is not for 1.0.0.Final so the merge can wait till you may want to check it locally as well... Thanks

[gsmet]

Thanks!