Skip to content

Fix http connection #156

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

Merged
merged 4 commits into from
May 27, 2021
Merged

Fix http connection #156

merged 4 commits into from
May 27, 2021

Conversation

IrishWhiskey
Copy link
Contributor

@IrishWhiskey IrishWhiskey commented May 12, 2021

Fix the rest implementation bug discussed in #155

Signed-off-by: Vincenzo Territo <[email protected]>
Signed-off-by: Vincenzo Territo <[email protected]>
@IrishWhiskey IrishWhiskey changed the title Http connection Fix http connection May 12, 2021
@thjaeckle thjaeckle added this to the 2.0.1 milestone May 17, 2021
@ffendt
Copy link
Contributor

ffendt commented May 25, 2021

Hi @IrishWhiskey,

thank you very much for your PR. I'll have a look at it and get back to you (sorry for taking so long).

@ffendt
Copy link
Contributor

ffendt commented May 26, 2021

Hi @IrishWhiskey,

your changes look very good. I'm just additionally adapting the ProxyAgent to also work without tls, then this is ready to merge. Good job, thank you 👍

@ffendt
Copy link
Contributor

ffendt commented May 26, 2021

@IrishWhiskey, can you verify if the rest implementation still works on your side?

@IrishWhiskey
Copy link
Contributor Author

Hi @ffendt. It works for me. Thanks for helping me :)

ffendt added a commit to bosch-io/ditto-clients that referenced this pull request May 27, 2021
@yufei-cai yufei-cai merged commit 10f0816 into eclipse-ditto:master May 27, 2021
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.

4 participants