Skip to content

fix(audit): resolve 6 critical bugs + standardize project files - #23

Merged
OneByJorah merged 6 commits into
mainfrom
audit/2026-07-05
Jul 6, 2026
Merged

fix(audit): resolve 6 critical bugs + standardize project files#23
OneByJorah merged 6 commits into
mainfrom
audit/2026-07-05

Conversation

@OneByJorah

Copy link
Copy Markdown
Owner

🔴 Critical Bugs Fixed

# Bug Fix
C1 rate_limiter.py dict init =() (would crash) Changed to ={}
C2 zammad.py missing import requests (NameError at runtime) Added import
C3 session_manager.py SQL injection via .format() Parameterized MAKE_INTERVAL query
C4 health_monitor.py Redis URL used PostgreSQL port 5432 Fixed to 6379
C5 whatsapp_webhook.py same Redis URL port error Fixed to 6379
C6 email_fetcher.py posted to non-existent /tickets/create Routed through /chat instead

📋 Standard Files Added

  • .dockerignore — build context optimization
  • j1.yaml — pipeline registry metadata
  • CHANGELOG.md — release history
  • .github/CODEOWNERS — PR ownership routing

📊 Score

73 → 82/100 (+9 points)

dependabot Bot and others added 6 commits July 6, 2026 01:42
Bumps [requests](https://github.com/psf/requests) from 2.32.3 to 2.33.0.
- [Release notes](https://github.com/psf/requests/releases)
- [Changelog](https://github.com/psf/requests/blob/main/HISTORY.md)
- [Commits](psf/requests@v2.32.3...v2.33.0)

---
updated-dependencies:
- dependency-name: requests
  dependency-version: 2.33.0
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [python-multipart](https://github.com/Kludex/python-multipart) from 0.0.20 to 0.0.31.
- [Release notes](https://github.com/Kludex/python-multipart/releases)
- [Changelog](https://github.com/Kludex/python-multipart/blob/main/CHANGELOG.md)
- [Commits](Kludex/python-multipart@0.0.20...0.0.31)

---
updated-dependencies:
- dependency-name: python-multipart
  dependency-version: 0.0.31
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [python-dotenv](https://github.com/theskumar/python-dotenv) from 1.0.1 to 1.2.2.
- [Release notes](https://github.com/theskumar/python-dotenv/releases)
- [Changelog](https://github.com/theskumar/python-dotenv/blob/main/CHANGELOG.md)
- [Commits](theskumar/python-dotenv@v1.0.1...v1.2.2)

---
updated-dependencies:
- dependency-name: python-dotenv
  dependency-version: 1.2.2
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
C1: rate_limiter dict typo fix. C2: zammad import requests. C3: session_manager SQL injection fix. C4/C5: Redis URL fixes. C6: email_fetcher endpoint fix. Added .dockerignore, j1.yaml, CHANGELOG.md, CODEOWNERS.
@OneByJorah
OneByJorah merged commit db797d3 into main Jul 6, 2026
4 of 5 checks passed
@OneByJorah
OneByJorah deleted the audit/2026-07-05 branch July 6, 2026 05:46
# Try to extract email from message
import re
email_match = re.search(r'[\w.+-]+@[\w-]+\.[\w.-]+', text)
email_match = re.search(r"[\w.+-]+@[\w-]+\.[\w.-]+", text)
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.

3 participants