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

add delay between bettercap and pwnagotchi starts #76

Merged
merged 1 commit into from
Nov 24, 2023

Conversation

noncenz
Copy link

@noncenz noncenz commented Nov 24, 2023

Add a delay as a temporary fix to a race condition between these two services starting

Description

Adds a 2 second sleep before pwnagotchi service starts, to delay deletion of the startup mode override file.

Motivation and Context

This is a temporary fix to issue #61 so that the manual mode caplet runs when appropriate.

  • [ X] I have raised an issue to propose this change (required)

How Has This Been Tested?

  • Tested manually on a Pi Zero 2W with wifi chipset version 1.
  • Tested with v1.7.9 and v.1.8.0 releases

To test:

  • Initiate a bluetooth tether to the pwnagotchi with no USB or ethernet connected.
  • Tail daemon.log to observe the ip address that the Bettercap service comes up on.
  • Toggle between manual and auto modes with the web ui.
  • Expect 127.0.0.1 when in auto mode and 0.0.0.0 when in manual.

Types of changes

  • [ X] Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • [ X] My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • [ X] I've read the CONTRIBUTION guide
  • [ X] I have signed-off my commits with git commit -s

Add a delay as a temporary fix to a race condition between these two services starting
@aluminum-ice aluminum-ice merged commit 56e3f3b into aluminum-ice:master Nov 24, 2023
@aluminum-ice aluminum-ice mentioned this pull request Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants