arades79 / hyperdome

the safest place to reach out
Other
27 stars 1 forks source link

Unused code cleanup #66

Closed arades79 closed 4 years ago

arades79 commented 4 years ago

Summary

Large purge of Onionshare code that is no longer used, as well as some cleanup in files that started from Onionshare.

All code was also re-formatted using black now, a future commit should add a pre-commit hook for black, and whenever CI/CD is worked out black -check will be included.

Some bandit lint notes were addressed as well to ensure the utmost in security. Namely all random operations were moved to use the secrets package for its cryptographic number generation.

No logic changes were made, and the app performs identically to how it did before, only with many unused code paths eliminated.

This partially addresses #12 which is an ongoing issue. Remaining items include changing icons, license files, and other meta-level vestige code to make the whole project bifurcated from Onionshare.

Checklist