Format Python code with psf/black push by github-actions[bot] · Pull Request #89 · mrayanasim09/python-projects

github-actions · 2026-08-03T06:59:09Z

There appear to be some python formatting errors in ac32097. This pull request
uses the psf/black formatter to fix these issues.

Summary by Sourcery

Enhancements:

Apply Black formatting to test utility modules without changing test logic or behavior.

This change is

#85) # Security Fixes: Resolve All Critical and High-Priority Issues ## Summary This PR addresses **all 50 security issues** identified in the Bandit security scan, reducing the count to **0 active issues**. ## Changes Made ### 🔴 Critical Fixes (2 issues) - **B324: MD5 Hash Usage** in `password.py` and `password_hash.py` - Added `bcrypt` support as the secure default for password hashing - Kept MD5 for educational purposes only, with explicit warnings - MD5 is cryptographically broken and should not be used for security ### 🟡 High Priority Fixes (9 issues) - **B113: Requests Without Timeout** in `connectivity.py`, `github.py`, `url.py` - Added 10-second timeout to all HTTP requests - Prevents hanging requests and potential DoS - **B607, B603, B404: Subprocess Security** in `network.py` - Added explicit `shell=False` parameter to all subprocess calls - Added Windows platform check (tool only works on Windows) - Improved error handling and user feedback ### 🟡 Medium Priority Fixes (4 issues) - **B105: Hardcoded Sensitive Paths** - `password.py`: Changed hardcoded empty string to user input - `password_manager.py`: Added environment variable support for file paths - **B110: Empty Except Block** in `notepad.py` - Replaced silent `except: pass` with proper error logging - Added error messages for file operations ### 🟢 Low Priority Fixes - **File Naming Issues** - Renamed `Calculator/ASCII .py` → `Calculator/ASCII.py` (removed space) - Renamed `Calculator/time_calulator.py` → `Calculator/time_calculator.py` (fixed typo) - Updated `README.md` references - **False Positives (B311)** - Added `# nosec B311` comments to 15+ game files - Random module usage in games is for gameplay, not security - These are intentional and safe uses of randomness ## Security Scan Results ### Before ``` Total Issues: 50 - B311 (random): 35 issues - B113 (timeout): 4 issues - B105 (hardcoded): 3 issues - B607/B603 (subprocess): 4 issues - B324 (MD5): 2 issues - B404 (subprocess import): 1 issue - B110 (empty except): 1 issue ``` ### After ``` Total Issues: 0 - 42 potential issues properly marked as intentional with # nosec comments - All Python files compile successfully ``` ## Testing - ✅ All 87 Python files compile without errors - ✅ Bandit security scan: 0 issues - ✅ No breaking changes to functionality - ✅ Backward compatible (except for renamed files) ## Files Modified - 25 files changed - 2 files renamed - 468 insertions, 290 deletions ## Impact - **Security**: Significantly improved - all critical/high issues resolved - **Functionality**: Preserved - all projects continue to work - **Maintainability**: Improved - better error handling and code quality Closes security scan findings from Bandit analysis. ## Summary by Sourcery Resolve Bandit-reported security issues by hardening password handling, network utilities, and HTTP usage while explicitly marking intentional randomness in games. New Features: - Add bcrypt-based password hashing and verification support for secure password storage and checking. Bug Fixes: - Add configurable timeouts and robust error handling to HTTP requests in connectivity, GitHub analysis, and URL utilities to prevent hangs and improve resilience. - Harden subprocess usage in the Windows network password retriever by disabling shell execution, adding platform checks, and improving error reporting. - Replace hardcoded password storage paths with environment-configurable files in the password manager to avoid sensitive hardcoded paths. - Replace silent exception handling in the GUI notepad with explicit error reporting for icon loading and file operations. Enhancements: - Allow configuring password and master key file locations via environment variables in the password manager. - Clarify and document the limited, educational use of MD5 while steering users toward bcrypt for secure password hashing. - Improve user feedback messages across utilities, including clearer errors for invalid URLs, file issues, and GitHub connectivity problems. - Standardize main entry points and structure for several utilities to support safer imports and reuse. Documentation: - Update README links to reflect corrected calculator file naming. Chores: - Rename calculator modules to fix naming and typographical issues and align README references. - Annotate random usage in game and utility scripts with Bandit `# nosec B311` markers where randomness is used purely for gameplay or non-security purposes.  - - - This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/mrayanasim09/python-projects/85)

vercel · 2026-08-03T06:59:12Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Updated (UTC)
100-python-projects	Error	Aug 3, 2026 7:04am
python-projects	Error	Aug 3, 2026 7:04am
python-projects-1	Error	Aug 3, 2026 7:04am

codesandbox · 2026-08-03T06:59:13Z

Review or Edit in CodeSandbox

Open the branch in Web Editor • VS Code • Insiders

Open Preview

sourcery-ai · 2026-08-03T06:59:16Z

Reviewer's Guide

Formats Utilities/test_utilities.py with Black, standardizing whitespace, string quoting, and line breaking without changing test logic or behavior.

File-Level Changes

Change	Details	Files
Reformat test module to Black style while preserving all test behavior.	Normalize blank lines and indentation throughout all test classes and methods Reflow long expressions, function calls, and assertions onto multiple lines per Black’s line-length rules Convert single-quoted strings (including decorators and main guard) to double-quoted strings where Black applies this rule Reformat dictionary literals and multi-line data structures with one key/value per line and trailing commas as needed Standardize import usage and inline helper function formatting without altering their logic	`Utilities/test_utilities.py`

Tips and commands

Interacting with Sourcery

Trigger a new review: Comment @sourcery-ai review on the pull request.
Continue discussions: Reply directly to Sourcery's review comments.
Generate a GitHub issue from a review comment: Ask Sourcery to create an
issue from a review comment by replying to it. You can also reply to a
review comment with @sourcery-ai issue to create an issue from it.
Generate a pull request title: Write @sourcery-ai anywhere in the pull
request title to generate a title at any time. You can also comment
@sourcery-ai title on the pull request to (re-)generate the title at any time.
Generate a pull request summary: Write @sourcery-ai summary anywhere in
the pull request body to generate a PR summary at any time exactly where you
want it. You can also comment @sourcery-ai summary on the pull request to
(re-)generate the summary at any time.
Generate reviewer's guide: Comment @sourcery-ai guide on the pull
request to (re-)generate the reviewer's guide at any time.
Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
pull request to resolve all Sourcery comments. Useful if you've already
addressed all the comments and don't want to see them anymore.
Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
request to dismiss all existing Sourcery reviews. Especially useful if you
want to start fresh with a new review - don't forget to comment
@sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

Enable or disable review features such as the Sourcery-generated pull request
summary, the reviewer's guide, and others.
Change the review language.
Add, remove or edit custom review instructions.
Adjust other review settings.

Getting Help

Contact our support team for questions or feedback.
Visit our documentation for detailed guides and information.
Keep in touch with the Sourcery team by following us on X/Twitter, LinkedIn or GitHub.

codacy-production · 2026-08-03T06:59:49Z

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

_{NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer}
_{TIP This summary will be updated as you push new changes.}

sourcery-ai

Hey - I've reviewed your changes and they look great!

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

Signed-off-by: Muhammad Rayyan Asim <mrayanasim09@gmail.com>

github-actions Bot requested review from drik-493 and mrayanasim09 as code owners August 3, 2026 06:59

pull-request-size Bot added the size/L label Aug 3, 2026

vercel Bot had a problem deploying to Preview – python-projects-1 August 3, 2026 06:59 Failure

vercel Bot had a problem deploying to Preview – 100-python-projects August 3, 2026 06:59 Failure

vercel Bot had a problem deploying to Preview – python-projects August 3, 2026 06:59 Failure

sourcery-ai Bot reviewed Aug 3, 2026

View reviewed changes

mrayanasim09 and others added 2 commits August 3, 2026 12:03

Update README.md

9364bee

Signed-off-by: Muhammad Rayyan Asim <mrayanasim09@gmail.com>

🎨 Format Python code with psf/black

5fb874a

github-actions Bot force-pushed the actions/black branch from 9edf32a to 5fb874a Compare August 3, 2026 07:03

pull-request-size Bot added size/M and removed size/L labels Aug 3, 2026

vercel Bot had a problem deploying to Preview – 100-python-projects August 3, 2026 07:03 Failure

vercel Bot had a problem deploying to Preview – python-projects August 3, 2026 07:04 Failure

vercel Bot had a problem deploying to Preview – python-projects-1 August 3, 2026 07:04 Failure

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Format Python code with psf/black push - #89

Format Python code with psf/black push#89
github-actions[bot] wants to merge 3 commits into
writing-unit-tests-37be7from
actions/black

github-actions Bot commented Aug 3, 2026 •

edited by mrayanasim09

Loading

Uh oh!

vercel Bot commented Aug 3, 2026 •

edited

Loading

Uh oh!

codesandbox Bot commented Aug 3, 2026

Uh oh!

sourcery-ai Bot commented Aug 3, 2026 •

edited

Loading

Interacting with Sourcery

Customizing Your Experience

Getting Help

Uh oh!

codacy-production Bot commented Aug 3, 2026

Uh oh!

sourcery-ai Bot left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

github-actions Bot commented Aug 3, 2026 • edited by mrayanasim09 Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by Sourcery

Uh oh!

vercel Bot commented Aug 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

codesandbox Bot commented Aug 3, 2026

Review or Edit in CodeSandbox

Uh oh!

sourcery-ai Bot commented Aug 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Reviewer's Guide

File-Level Changes

Interacting with Sourcery

Customizing Your Experience

Getting Help

Uh oh!

codacy-production Bot commented Aug 3, 2026

Up to standards ✅

Uh oh!

sourcery-ai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

github-actions Bot commented Aug 3, 2026 •

edited by mrayanasim09

Loading

vercel Bot commented Aug 3, 2026 •

edited

Loading

sourcery-ai Bot commented Aug 3, 2026 •

edited

Loading