feat: Update pagination to use reactive + various bug fixes and upgrades

• feat: Update pagination to use reactive + various bug fixes and upgrades

requested review from @cguindon, @malowe, and @zacharysabourin

assigned to @zacharysabourin

requested changes

added 1 commit

• 7c179608 - feat: Integrate async pagination + update imperative pagination to use RestResponse

Compare with previous version

mentioned in merge request eclipsefdn-working-groups-api!79 (merged)

added 1 commit

• b91de155 - fix: Updat configs

Compare with previous version

added 1 commit

• 1e6ff03d - feat: Add testing for MockDataPaginationHandler

Compare with previous version

added 1 commit

• 45cdb0bb - feat: Convert handler to return a RestResponse + update tests to check headers

Compare with previous version

requested review from @malowe

marked this merge request as ready

resolved all threads

The only question I have is did you confirm this build still handles GZIP'd content? It looks like you removed all of the GZIP properties from the property files.

I did! I tested this along with this patch: eclipsefdn-working-groups-api!79 (merged)

It can be confirmed by using the GzipClientRequestFilter to enable GZIP'd content through RestClients and setting the quarkus.http.enable-compression config to false. This will cause it to fail (due to being unable to properly deserialize the GZIP content) until the config has been re-enabled.

I removed the tests and configs for this since we no longer need to handle to decompression manually. We just need to ensure the @Compressed annotation is set on our Restclients and that quarkus.http.enable-compression is set to true.

approved this merge request

merged