David Radley

@grzegorz8 thanks for your support on this. On Result pagination: - I think it depends on the use cases we want to go after . - Use case 1) I...

@kristoffSC The fix isn't quite ready so I let the pr in draft. I was going to get the code clean before making the pr ready for review. But it...

@kristoffSC pushed up the new code addressing most of the code review comments - testing now.

@kristoffSC Hi I have been testing against the watsonx APIs, the latest fix is not working (I will sort that out). During this testing I found that the SSL handshake...

@kristoffSC the new design is not working because the preprocessing only works if there is a header to preprocess. In my case it was not working as I had not...

I have included the public certs and it works. By this I mean while testing I put the public server cert in an ssl folder and referred to it in...

Hi @kristoffSC, In response to your comment > Regarding public certs -> I've replayed in [#119 (comment)](https://github.com/getindata/flink-http-connector/issues/119#issuecomment-2323042815) > Regarding this Pr here, I also think option 3 is good, but...

@grzegorz8 Hi there I have fixed up the requested changes- please could you review and ee if there is anything else you can spot and GTM if you are good...

> > @grzegorz8 Hi there I have fixed up the requested changes- please could you review and ee if there is anything else you can spot and GTM if you...

@grzegorz8 On the review comment , saying it was not addressed; I do not see any format changes now introduced by the fix. I see : ![Screenshot 2024-10-10 at 11...