Skip to content

Conversation

@s0lst1c3
Copy link

The following updates have been made:

  • Fixed indentation errors caused by mixing tabs and spaces. Project now uses spaces only.
  • Fixed import statements to work with modern versions of Scapy.
  • Removed unused imports.
  • Psyco is no longer maintained and does not support Python 2.7. Removed from project in favor of PyPy.
  • Code cleanup and optimizations.
  • Project now uses the newer argparse module instead of getopt to parse command line arguments.
  • Input validation is now done through argparse as much as possible.
  • Added utils.py file containing commonly used utility functions.
  • Added include guard to all files.
  • Added pip requirements file.
  • Updated README.

…spaces only. Fixed import statements to work with modern versions of Scapy. Removed unused imports. Psyco is no longer maintained and does not support Python 2.7. Removed from project in favor of PyPy. Code cleanup and optimizations. Project now uses the newer argparse module instead of getopt to parse command line arguments. Input validation is now done through argparse as much as possible. Added utils.py file containing commonly used utility functions. Added include guard to all files. Added pip requirements file. Updated README.
@gpotter2
Copy link

gpotter2 commented Jun 4, 2019

Hi ! Any news about this ? Looks alright to me

The project looks pretty dead, and we were linking it from our README 😅 If this PR cannot be merged (inactive maintainers), we'll just drop it.

@s0lst1c3
Copy link
Author

s0lst1c3 commented Jun 4, 2019

@gpotter2 If GDS Labs is no longer interested in maintaining it, I have a fork on my personal Github you can use. Just let me know and I'll put it on my list of of things to actively maintain.

@gpotter2
Copy link

gpotter2 commented Jun 5, 2019

Thanks for the call.

I don't think many people still use wifitap: You could probably choose to maintain it yourself, but I'm not sure if it's worth the effort :/ It's up to you

We've ended up keeping the link for a historical purpose.

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