quarkusio / quarkus-quickstarts

Quarkus quickstart code
https://quarkus.io
Apache License 2.0
1.92k stars 1.43k forks source link

Use OidcClientFilter and AccessToken annotations #1402

Closed sberyozkin closed 2 months ago

sberyozkin commented 3 months ago

Avoiding the direct filter registration, as will also be shown in the updated quickstart doc

quarkus-bot[bot] commented 3 months ago

:waning_crescent_moon: This workflow status is outdated as a new workflow run has been triggered.

Status for workflow Pull Request Build - development

This is the status report for running Pull Request Build - development on commit 00f0b9fb7ca13a7dce4375ddb4db83dae7492fd5.

Failing Jobs

Status Name Step Failures Logs Raw logs
Build - JDK 17 Build with Maven Failures Logs Raw logs

Full information is available in the Build summary check run.

Failures

:gear: Build - JDK 17 #

- Failing: hibernate-orm-multi-tenancy-quickstart quartz-quickstart security-openid-connect-multi-tenancy-quickstart 

:package: hibernate-orm-multi-tenancy-quickstart

org.acme.hibernate.orm.FruitsEndpointTest.testListAllFruitsBaseSchema - More details - Source on GitHub

``` java.lang.RuntimeException: java.lang.RuntimeException: Failed to start quarkus at io.quarkus.test.junit.QuarkusTestExtension.throwBootFailureException(QuarkusTestExtension.java:643) at io.quarkus.test.junit.QuarkusTestExtension.interceptTestClassConstructor(QuarkusTestExtension.java:727) at java.base/java.util.Optional.orElseGet(Optional.java:364) at java.base/java.util.ArrayList.forEach(ArrayList.java:1511) at java.base/java.util.ArrayList.forEach(ArrayList.java:1511) Caused by: java.lang.RuntimeException: Failed to start quarkus at io.quarkus.runner.ApplicationImpl.doStart(Unknown Source) ```

:package: quartz-quickstart

org.acme.quartz.TaskResourceTest.tasks - More details - Source on GitHub

``` java.lang.RuntimeException: java.lang.RuntimeException: Failed to start quarkus at io.quarkus.test.junit.QuarkusTestExtension.throwBootFailureException(QuarkusTestExtension.java:643) at io.quarkus.test.junit.QuarkusTestExtension.interceptTestClassConstructor(QuarkusTestExtension.java:727) at java.base/java.util.Optional.orElseGet(Optional.java:364) at java.base/java.util.ArrayList.forEach(ArrayList.java:1511) at java.base/java.util.ArrayList.forEach(ArrayList.java:1511) Caused by: java.lang.RuntimeException: Failed to start quarkus at io.quarkus.runner.ApplicationImpl.doStart(Unknown Source) ```

:package: security-openid-connect-multi-tenancy-quickstart

org.acme.quickstart.oidc.CodeFlowTest.testReAuthenticateWhenSwitchingTenants line 89 - More details - Source on GitHub

``` org.opentest4j.AssertionFailedError: expected: but was: <> at org.junit.jupiter.api.AssertionFailureBuilder.build(AssertionFailureBuilder.java:151) at org.junit.jupiter.api.AssertionFailureBuilder.buildAndThrow(AssertionFailureBuilder.java:132) at org.junit.jupiter.api.AssertEquals.failNotEqual(AssertEquals.java:197) at org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:182) at org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:177) at org.junit.jupiter.api.Assertions.assertEquals(Assertions.java:1145) at org.acme.quickstart.oidc.CodeFlowTest.testReAuthenticateWhenSwitchingTenants(CodeFlowTest.java:89) ```
sberyozkin commented 3 months ago

There is an unrelated OIDC quickstart failure which I'll look at shortly

quarkus-bot[bot] commented 2 months ago

Status for workflow Pull Request Build - development

This is the status report for running Pull Request Build - development on commit 1f2c25ed58b6e39617084b20ed4115156b5060c5.

:white_check_mark: The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.