-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Fix MP REST 1.4 TCK #7639
Merged
Merged
Fix MP REST 1.4 TCK #7639
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
gsmet
approved these changes
Mar 6, 2020
mkouba
approved these changes
Mar 6, 2020
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Do we have a tracking issue to replace the workaround and a corresponding RESTEasy issue?
This was referenced Mar 6, 2020
Thanks @stuartwdouglas. Is Windows JDK 8 CI failure a known issue? |
asoldano
pushed a commit
to resteasy/resteasy
that referenced
this pull request
Mar 10, 2020
* Fix for Quarkus (quarkusio/quarkus#7639) Signed-off-by:Phillip Kruger <[email protected]> * Added check for CDI in construct. Signed-off-by:Phillip Kruger <[email protected]> * Checkstyle fixes Signed-off-by:Phillip Kruger <[email protected]>
ronsigal
added a commit
to resteasy/resteasy
that referenced
this pull request
Jun 16, 2020
* 4.5.0-SNAPSHOT -> 4.5.0.Final (#2307) * 4.5.0.Final -> 4.6.0-SNAPSHOT * [RESTEASY-2496]:Add test case * [RESTEASY-2510] Split RESTUtils to links provider and injector * RESTEASY-2445 * [RESTEASY-2517] Async io fixes (#2315) * Apply same fix for SSE streams into raw streams To avoid onNext finishing after onComplete * Test for raw stream and async io * Fixed typo in onComplete/onWriteComplete * Fix potential bug spotted by Stuart * [RESTEASY-2520] CDI BeanManager fix for Quarkus (#2318) * Fix for Quarkus (quarkusio/quarkus#7639) Signed-off-by:Phillip Kruger <[email protected]> * Added check for CDI in construct. Signed-off-by:Phillip Kruger <[email protected]> * Checkstyle fixes Signed-off-by:Phillip Kruger <[email protected]> * [RESTEASY-2518] added value check and test for query param replacement * [RESTEASY-2503] Throw ViolationException for EJBs with just parameter violations. * RESTEASY-2519 (#2320) * RESTEASY-2519 * update * [RESTEASY-2535] Upgrade Jackson2 to 2.10.3 * [RESTEASY-2541] Fix RestClientListeners * [RESTEASY-2542] Excluding tests * RESTEASY-2554 Update section "Upgrading RESTEasy within WildFly" (#2349) * update * RESTEASY-2554 * remove module * update doc * minor fix * minor addition * minor fix * minor addition * fix version * minor fix * [RESTEASY-2559] Improper validation of response header in MediaTypeHeaderDelegate.java class * [RESTEASY-2551] Upgraded io.netty:netty-all to version 4.1.48.Final * [RESTEASY-2543] add regex change * [RESTEASY-2226]:NullPointerException in PatchMethodFilter * [RESTEASY-2385] Added support for embedded MultipartOutput types. Added testcase. * [RESTEASY-2480] re-enable code handling base64 * [RESTEASY-2532] replace reference to jakarta-json * [RESTEASY-2565] Mp client async interceptors (#2359) * Tie the AsyncInvocationInterceptor to each invocation * Spotted a bug, I think * [RESTEASY-2566] Set response code to 500 if writing exception fails (#2358) * [RESTEASY-2562] Incorrect status code when WebApplicationException is… (#2365) * [RESTEASY-2562] Incorrect status code when WebApplicationException is thrown while parsing @*Param * [RESTEASY-2562] Checkstyle - make arg final * [RESTEASY-2495] HeaderParam could have a ParamConverter related (#2292) AbstractInvocationCollectionProcessor is processing Array and Collection of elements being ParamConverter aware. But with single values is delegating in final class to process the value. HeaderParamProcessor is not being aware of such transformation. * [RESTEASY-2495] made HeaderParamProcessor aware of ParamConverter and provided testcase * [RESTEASY-2526] Fix by using asyncutil lib for async loops (#2368) * [RESTEASY-2466] removed profile arquillian.remote * [RESTEASY-2531] expanded and refactored chapter * [RESTEASY-2544] pom updates to arquillian-util and integration-tests * [RESTEASY-2542] ignore any IOException in SSEOutputStream.close() * Test against WildFly 19.0.0.Final instead of Beta2 * [RESTEASY-2526] Turn the com.ibm.async.asyncutil module into private * [RESTEASY-2550] Skip null parameters being processed by ParamConverter's. Also throw an NPE if a parameter annotated with @PathParam it not allowed. https://issues.redhat.com/browse/RESTEASY-2550 * [RESTEASY-2542] Restore tests * [RESTEASY-2239] updated mime4j to version 0.8.2 * Add GitHub Action for running CI builds * [RESTEASY-2561] Allow tests to run in environments other than a Linux local environment. * [RESTEASY-2564] Use a single thread executor instead of the default executor for a more graceful shutdown. * Remove TravisCI runs duplicated by Github Actions * [RESTEASY-2542] Exclude a test again, still failing * [RESTEASY-2542] Exlcude the whole SseTest, not just testReconnect() * Only upload archives for failed runs. Also fix the name to not end with ".zip" * [RESTEASY-2548] add new provider with higher priority than JaxrsFormProvider adding test * [RESTEASY-2560] set scope for must junit dependencies to test * [RESTEASY-2391] changed code to use getMethod() * [RESTEASY-2472]:@suspended AsyncResponse timeout still runs, even if … (#2372) * [RESTEASY-2472]:@suspended AsyncResponse timeout still runs, even if the response was resumed before * Update AsyncTimeoutResource.java * Update AsyncTimeoutResource.java Bogus change to rerun tests. Co-authored-by: Ron Sigal <[email protected]> * [RESTEASY-2563: SseBroadcasterImpl.close() puts notifyOnCloseListeners(eventSink) in finally block. (#2396) * [RESTEASY-2576] added client modules * [RESTEASY-2582] CompletionStageResponseTest intermittent failures (#2390) * Prevent memory visibility issues * Refactor AsyncResponseCallback to isolate data from different tests * Same sleep time for similar methods * [RESTEASY-2571] Avoid a NullPointerException of the MediaType associated with request headers is null. https://issues.redhat.com/browse/RESTEASY-2571 * [RESTEASY-2569] RestClient does not propogate incoming headers with inherited Interface * [RESTEASY-2582] Add logs to the test * [RESTEASY-2582] Fix race condition in test * [RESTEASY-1203] Fix validation for JNDI resources. (#2406) * [RESTEASY-2542][RESTEASY-2585] Workaround SseTest intermittent failures (#2404) * Enable SseTest * [RESTEASY-2542]:Fix Sse reconnect test * [RESTEASY-2542]:More Sse test fix * Fix typo and add comment * [RESTEASY-2017]:Fix SSE doesn't work with inherited annotations * Updated links to the new project site * [RESTEASY-2588] Upgrade jackson to 2.10.4 * [RESTEASY-2587] Update classmate to 1.5.1 * Fix RESTEASY-2573 by using ResteashContext.addCloseable variants * Test RESTEASY-2573 for SSE/raw/collect async streaming * [RESTEASY-2542] wait for proxy to be started * [RESTEASY-2576] Added modules to resteasy-bom/pom.xml (#2417) * Added note to testsuite/README.MD on how to run single microprofile-tck tests (#2426) * SseEventOutputImpl: avoid deadlock by avoiding getting into critical sections early RESTEASY-2585 * HttpServletResponseWrapper: try working around UNDERTOW-1713: RESTEASY-2585 * RESTEASY-2524 (#2332) * RESTEASY-2524 * trailing spaces removed * [RESTEASY-2585]:Remove Thread.sleep() workaround in SseTest * Updating target test containers * [RESTEASY-2598] Upgrade Infinispan to 10.1.8 * [RESTEASY-2598] Exclude ServerCacheInterceptorTest in WFLY19 * Fix modules test references to properly run with WFLY20 too * Exclude test failing on GithubActions CI due to microprofile/microprofile-rest-client#265 * [RESTESY-2601] Exclude test * [RESTEASY-2605] Remove the fallback to URLConnection when SSL is disabled for REST Client (#2431) The fallback was originally added to get around limitation in Apache HTTP Client when SSL is completely disabled in native binaries. However we can get around these limitations in a more elegant manner in Quarkus (see quarkusio/quarkus#9773). By defaulting to Apache HTTP Client, things like HTTP Patch can be supported OOTB (see quarkusio/quarkus#9342). Co-authored-by: Ron Sigal <[email protected]> Co-authored-by: Jim Ma <[email protected]> Co-authored-by: Gytis Trikleris <[email protected]> Co-authored-by: 阿男 <[email protected]> Co-authored-by: Stéphane Épardaud <[email protected]> Co-authored-by: Phillip Krüger <[email protected]> Co-authored-by: R Searls <[email protected]> Co-authored-by: Bartosz Spyrko-Smietanko <[email protected]> Co-authored-by: Stuart Douglas <[email protected]> Co-authored-by: Anil Gursel <[email protected]> Co-authored-by: David Santos <[email protected]> Co-authored-by: James Perkins <[email protected]> Co-authored-by: Duncan <[email protected]> Co-authored-by: bobbyphilip <[email protected]> Co-authored-by: Tommaso Borgato <[email protected]> Co-authored-by: Georgios Andrianakis <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.