Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Execute ClientResponseFilter as part of the abort chain in JAX-RS Client #16731

Merged
merged 1 commit into from
Apr 23, 2021

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Apr 22, 2021

This doesn't seem to be specified by the spec or properly tested in the TCK
(I actually had to disable a single test that threw a NPE), but it does
seem to be what RESTEasy does

Relates to: #16702

This doesn't seem to be specified by the spec or properly tested in the TCK
(I actually had to disable a single test that threw a NPE), but it does
seem to be what RESTEasy does

Relates to: quarkusio#16702
@quarkus-bot
Copy link

quarkus-bot bot commented Apr 22, 2021

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

Failing Jobs - Building 6889b6d

⚠️ Artifacts of the workflow run were not available thus the report misses some details.

Status Name Step Test failures Logs Raw logs
Initial JDK 11 Build ⚠️ Check → Logs Raw logs
Attach pull request number ⚠️ Check → Logs Raw logs
CI Sanity Check ⚠️ Check → Logs Raw logs
🚫 Devtools Tests - JDK ${{ matrix.java.name }}
🚫 Devtools Tests - JDK 11 Windows
🚫 Gradle Tests - JDK 11 ${{ matrix.os.family }}
🚫 JVM Tests - JDK ${{ matrix.java.name }}
🚫 JVM Tests - JDK 11 Windows
🚫 Maven Tests - JDK ${{ matrix.java.name }}
🚫 Maven Tests - JDK 11 Windows
🚫 MicroProfile TCKs Tests
🚫 Native Tests - ${{ matrix.category }}
Native Tests - Read JSON matrix ⚠️ Check → Logs Raw logs
🚫 Native Tests - Windows - ${{ matrix.category }}

@michalszynkiewicz michalszynkiewicz merged commit 1997498 into quarkusio:main Apr 23, 2021
@quarkus-bot quarkus-bot bot added this to the 2.0 - main milestone Apr 23, 2021
@geoand geoand deleted the #16702 branch April 23, 2021 09:24
@gsmet
Copy link
Member

gsmet commented Apr 26, 2021

This one wasn't backported either.

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

Successfully merging this pull request may close these issues.

3 participants