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

Make it simpler. #41

Closed
wants to merge 22 commits into from
Closed

Make it simpler. #41

wants to merge 22 commits into from

Conversation

byehack
Copy link

@byehack byehack commented Apr 26, 2023

As this is just PoW script and not recommended for production & personal use, let make it simpler for who want read the script:

  • removed unused imports
  • handle python-resources availability
  • don't use first_flag variable
  • move send_data_in_fragment to handshake
  • unpack functions from class
  • code polish
  • main work moved to handshake function
  • backend_connect for user specified rotating IPs

@byehack byehack marked this pull request as draft April 26, 2023 12:12
@byehack byehack marked this pull request as ready for review April 26, 2023 12:36
@cs127
Copy link

cs127 commented Apr 26, 2023

why remove the interpreter directive (#!/usr/bin/env python3)?

@byehack
Copy link
Author

byehack commented Apr 26, 2023

Thanks, added.

@byehack byehack marked this pull request as draft April 26, 2023 17:08
@byehack byehack marked this pull request as ready for review April 26, 2023 20:08
@byehack byehack mentioned this pull request Apr 27, 2023
@byehack
Copy link
Author

byehack commented Apr 27, 2023

#61

@byehack byehack closed this Apr 27, 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