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

Fix MP REST 1.4 TCK #7639

Merged
merged 1 commit into from
Mar 6, 2020
Merged

Fix MP REST 1.4 TCK #7639

merged 1 commit into from
Mar 6, 2020

Conversation

stuartwdouglas
Copy link
Member

No description provided.

@boring-cyborg boring-cyborg bot added area/dependencies Pull requests that update a dependency file area/rest-client labels Mar 6, 2020
@stuartwdouglas stuartwdouglas requested review from gsmet and mkouba March 6, 2020 11:51
Copy link
Contributor

@mkouba mkouba left a 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?

@gsmet gsmet added this to the 1.4.0 milestone Mar 6, 2020
@gsmet gsmet merged commit b36e24d into quarkusio:master Mar 6, 2020
@kenfinnigan
Copy link
Member

Thanks @stuartwdouglas.

Is Windows JDK 8 CI failure a known issue?

@gsmet gsmet modified the milestones: 1.4.0, 1.3.0.CR2 Mar 6, 2020
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
area/dependencies Pull requests that update a dependency file area/rest-client
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants