[GH-ISSUE #604] Feature proposal: option to remove web server #193

Closed
opened 2026-02-28 00:00:13 +03:00 by kerem · 12 comments
Owner

Originally created by @dwillmore on GitHub (Sep 14, 2017).
Original GitHub issue: https://github.com/SpacehuhnTech/esp8266_deauther/issues/604

Hello, all. I've been working with esp8266-deauther with an OLED display. In that use case, I have no need for the web UI. Would a patch to disable that portion of the code based on a config option be welcomed?

Originally created by @dwillmore on GitHub (Sep 14, 2017). Original GitHub issue: https://github.com/SpacehuhnTech/esp8266_deauther/issues/604 Hello, all. I've been working with esp8266-deauther with an OLED display. In that use case, I have no need for the web UI. Would a patch to disable that portion of the code based on a config option be welcomed?
kerem 2026-02-28 00:00:13 +03:00
Author
Owner

@tobozo commented on GitHub (Sep 14, 2017):

You can disable the WebUI from the menu by setting the WiFi off, is that what you mean ?

<!-- gh-comment-id:329578417 --> @tobozo commented on GitHub (Sep 14, 2017): You can disable the WebUI from the menu by setting the WiFi off, is that what you mean ?
Author
Owner

@dwillmore commented on GitHub (Sep 14, 2017):

I saw that and wondered what it did--for me all it does is drop the power consumption below the threshold that my USB supply thinks the attached device is done and powers off. So, that's been a convenient 'power device off'.

But, no, I was thinking of a #define that would disable inclusing of the web UI at build time.

<!-- gh-comment-id:329579883 --> @dwillmore commented on GitHub (Sep 14, 2017): I saw that and wondered what it did--for me all it does is drop the power consumption below the threshold that my USB supply thinks the attached device is done and powers off. So, that's been a convenient 'power device off'. But, no, I was thinking of a #define that would disable inclusing of the web UI at build time.
Author
Owner

@tobozo commented on GitHub (Sep 14, 2017):

Your suggestion is very helpful indeed.

I just did a quick test using something like #ifdef USE_WEBSERVER on my modified local version and found out this is a better workaround for an annoying bug we're trying to squash, see #208 and #587

Despite the fact it isn't on the roadmap, I'm quite sure this patch will be welcome.

<!-- gh-comment-id:329587805 --> @tobozo commented on GitHub (Sep 14, 2017): Your suggestion is very helpful indeed. I just did a quick test using something like `#ifdef USE_WEBSERVER` on my modified local version and found out this is a better workaround for an annoying bug we're trying to squash, see #208 and #587 Despite the fact it isn't on the roadmap, I'm quite sure this patch will be welcome.
Author
Owner

@dwillmore commented on GitHub (Sep 14, 2017):

Oh, how fortunate! Would you like me to make up a patch or--since you already seem to have done it--would you like to take this up? Would an option to build without serial support also be allowed?

I'm just looking for ways to use less program memory so that an OLED or similar display would work with a full MAC database on a 1MB board like the esp8285.

<!-- gh-comment-id:329590734 --> @dwillmore commented on GitHub (Sep 14, 2017): Oh, how fortunate! Would you like me to make up a patch or--since you already seem to have done it--would you like to take this up? Would an option to build without serial support also be allowed? I'm just looking for ways to use less program memory so that an OLED or similar display would work with a full MAC database on a 1MB board like the esp8285.
Author
Owner

@tobozo commented on GitHub (Sep 14, 2017):

Yes please submit a PR to the testing branch.

Just ignore the new branch it's only here as a POC.

<!-- gh-comment-id:329598286 --> @tobozo commented on GitHub (Sep 14, 2017): Yes please submit a PR to the testing branch. Just ignore the new [branch](https://github.com/spacehuhn/esp8266_deauther/tree/serverless) it's only here as a POC.
Author
Owner

@dwillmore commented on GitHub (Sep 14, 2017):

For the serial port disabling?

<!-- gh-comment-id:329606065 --> @dwillmore commented on GitHub (Sep 14, 2017): For the serial port disabling?
Author
Owner

@tobozo commented on GitHub (Sep 14, 2017):

I meant for the web server support

Disabling serial would be another feature request I guess, and probably subject to discussion too as it is a bit of an aggressive change towards the default behaviour of the tool.

<!-- gh-comment-id:329617712 --> @tobozo commented on GitHub (Sep 14, 2017): I meant for the web server support Disabling serial would be another feature request I guess, and probably subject to discussion too as it is a bit of an aggressive change towards the default behaviour of the tool.
Author
Owner

@dwillmore commented on GitHub (Sep 14, 2017):

Understood. I just thought I'd mention it. I'll open a different issue for that. I'll also go check out the testing branch and make the changes to that to remove the web UI.

<!-- gh-comment-id:329618046 --> @dwillmore commented on GitHub (Sep 14, 2017): Understood. I just thought I'd mention it. I'll open a different issue for that. I'll also go check out the testing branch and make the changes to that to remove the web UI.
Author
Owner

@dwillmore commented on GitHub (Oct 2, 2017):

Yeah, this one will be pretty huge. We might want to wait until after the next major revision.

<!-- gh-comment-id:333511403 --> @dwillmore commented on GitHub (Oct 2, 2017): Yeah, this one will be pretty huge. We might want to wait until after the next major revision.
Author
Owner

@tobozo commented on GitHub (Mar 24, 2018):

implemented in the next version, closing this

<!-- gh-comment-id:375908329 --> @tobozo commented on GitHub (Mar 24, 2018): implemented in the next version, closing this
Author
Owner

@tobozo commented on GitHub (Mar 31, 2018):

@dwillmore have you tried with the esp8285 since the new version is out ?

<!-- gh-comment-id:377673588 --> @tobozo commented on GitHub (Mar 31, 2018): @dwillmore have you tried with the esp8285 since the new version is out ?
Author
Owner

@dwillmore commented on GitHub (Mar 31, 2018):

@tobozo Nope, sorry, I was at a conference last Saturday when you were closing all those issues and releasing 2.0. Great work! I've been swampled since them, but I'll get around to checking it out. I'll check out the 'disable serial' option as well. Then I'll rebase my Quadrature Encoder UI code for 2.0 if that's all right?

<!-- gh-comment-id:377725540 --> @dwillmore commented on GitHub (Mar 31, 2018): @tobozo Nope, sorry, I was at a conference last Saturday when you were closing all those issues and releasing 2.0. Great work! I've been swampled since them, but I'll get around to checking it out. I'll check out the 'disable serial' option as well. Then I'll rebase my Quadrature Encoder UI code for 2.0 if that's all right?
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/esp8266_deauther#193
No description provided.