Skip to content

Conversation

@zendrix396
Copy link
Contributor

Closes #5

Added a CLI to make the script interactive. All the features from the issue are in.

  • You can now use flags like --login, --view-logs, and --setup. Run python wifi_auto_login.py --help to see everything.
  • Updated the setup.md with necessary instructions.

Copy link
Owner

@01bps 01bps left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@zendrix396
Kindly test it locally and share the output through image and/or clip.
Rest it LGTM

@zendrix396
Copy link
Contributor Author

So, the thing is I am outside and dont have access to wifi and respective captive portal, but it uses the same script so will work seamlessly. Rest other cases I have tested
Test: setup wizard
image

Connection test: Since the URL in the script is a placeholder, it will likely fail, which is a successful test in itself.
image

Test: See logs
image

Test: Clear logs
image

@01bps
Copy link
Owner

01bps commented Oct 2, 2025

Sure,take your time and get it verified once you are back,you send and recieve any parameter in case you don't have a captive portal.Testing it is necessary in anyways
Good work @zendrix396

@zendrix396
Copy link
Contributor Author

yeah actually I won't be having captive portal for a while so I tested printing the payload being send in a dry run as you can see below
Debugging
image
Output
image

I hope this will suffice, the cli argparse is just pinging the originally present wifi_login() function when someone sends the --login parameter, so it will work as expected if wifi_login() is working fine.
image

@zendrix396 zendrix396 force-pushed the feature/cli-interface branch from fdf6799 to 8763093 Compare October 2, 2025 16:41
@zendrix396
Copy link
Contributor Author

zendrix396 commented Oct 2, 2025

@01bps I've resolved the conflicts, please review and merge the changes

@01bps 01bps merged commit 01ea568 into 01bps:main Oct 2, 2025
@01bps
Copy link
Owner

01bps commented Oct 2, 2025

Good Work @zendrix396
LGTM, Merged!

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.

Add Command-Line Interface

2 participants