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

Support HTTPS webhook #48

Open
Seinuh opened this issue Nov 10, 2023 · 20 comments
Open

Support HTTPS webhook #48

Seinuh opened this issue Nov 10, 2023 · 20 comments
Labels
enhancement New feature or request

Comments

@Seinuh
Copy link

Seinuh commented Nov 10, 2023

Sending a webhook is only possible via HTTP. my cloud webhook address is based on https. Hopefully in a next update there will be support for both HTTP and HTTPS

@redakker
Copy link
Owner

Good point, I set this ticket as an enhancement.

@redakker redakker added the enhancement New feature or request label Nov 10, 2023
@redakker
Copy link
Owner

@Seinuh Just received your support. Thanks! This is the first time when somebody supports me in this way.
You made my day! I'm very thankful! <3

@redakker
Copy link
Owner

This is not that easy how i expected
Here is an article that needs to be tried. This is a note comment for the solution:
https://maakbaas.com/esp8266-iot-framework/logs/https-requests/

@Seinuh
Copy link
Author

Seinuh commented Nov 16, 2023

This is not that easy how i expected Here is an article that needs to be tried. This is a note comment for the solution: https://maakbaas.com/esp8266-iot-framework/logs/https-requests/

I see... because it frustrates me that v1.11 works stably for you, I have now installed a standalone mosquitto broker on a rasberry pi 😉🤞

@redakker
Copy link
Owner

No, https have never worked, I need to change the code for that based on that article.
This is a webhook ticket :) Probably you commented on the wrong ticket :)

@Seinuh
Copy link
Author

Seinuh commented Nov 16, 2023

No, https have never worked, I need to change the code for that based on that article. This is a webhook ticket :) Probably you commented on the wrong ticket :)
That's right, only http works.. but I created a new mqtt broker here to see if I can get more stability

@redakker
Copy link
Owner

Unfortunately the first POC is not working on my lab based on the article.
Need more investigations

@redakker redakker changed the title Webhook v1.09 Support HTTPS webhook Nov 17, 2023
@kohlthaver
Copy link

kohlthaver commented Feb 25, 2024

Hi,

any news on this? Just started to try out your code and after some debugging found out that my webhook forces https so I will either get just a 302 redirect (which the code doesn't follow) or a message "error on http request" which I suppose is related to https not working.

@redakker
Copy link
Owner

Somehow client.setInsecure(); is not working for me. I look around again, and let's see.

@Seinuh
Copy link
Author

Seinuh commented Apr 5, 2024

Any update for HTTPS webhook ?

@redakker
Copy link
Owner

Unfortunately, I had no time for that. Give me more time, please

@Seinuh
Copy link
Author

Seinuh commented Jan 8, 2025

@redakker any update for https webhook?

@redakker
Copy link
Owner

redakker commented Jan 8, 2025

@redakker any update for https webhook?

I'll take a look soon. I appreciate your patience.

@redakker
Copy link
Owner

redakker commented Jan 8, 2025

@Seinuh I've updated the project (library update, and https support). I did not do any tests, just i have a successful build.
So I'm not sure the functionality is working.
please do a test in a NON_PRODUCTION environment first.
Later ofcourse, I'll test for myself.

@Seinuh
Copy link
Author

Seinuh commented Jan 8, 2025

@redakker Nice i will test it, should be a major update for this project.

@Seinuh
Copy link
Author

Seinuh commented Jan 8, 2025

Hi,

any news on this? Just started to try out your code and after some debugging found out that my webhook forces https so I will either get just a 302 redirect (which the code doesn't follow) or a message "error on http request" which I suppose is related to https not working.

Maybe your can also test it... in latest bèta version

@Seinuh
Copy link
Author

Seinuh commented Jan 9, 2025

@redakker unfortunately i run into a invalid header by flashing my test esp32, is it possible to upload the latest (201) as a betà version on your web tool.. so I can flash via web tool

@redakker
Copy link
Owner

redakker commented Jan 9, 2025

@redakker unfortunately i run into a invalid header by flashing my test esp32, is it possible to upload the latest (201) as a betà version on your web tool.. so I can flash via web tool

@Seinuh What if you upload one of the available versions from the web tool, and then update on the device after with version 201?

@Seinuh
Copy link
Author

Seinuh commented Jan 9, 2025

@redakker unfortunately i run into a invalid header by flashing my test esp32, is it possible to upload the latest (201) as a betà version on your web tool.. so I can flash via web tool

@Seinuh What if you upload one of the available versions from the web tool, and then update on the device after with version 201?

I flashed 201 succesfully via https://esphome.github.io/esp-web-tools/ but it's not working hanging on "loading" so i reflashed it to 193 also via https://esphome.github.io/esp-web-tools/ and that one works for me...

So i think 201 has a bug, did you get it working ?

@Seinuh
Copy link
Author

Seinuh commented Jan 9, 2025

@redakker i did your option i flashed from web tool after that i update to 201 but after that it's hanging on loading

Schermafbeelding 2025-01-09 152358

So i think it's a bug in 201

@redakker did some changes and got 201 up and running stable but unfortunately only local webhook is working for me.

So my test ESP32 is up and running 🔥 and will test another build if it's available 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants