Skip to content

fix: remove hardcoded URLs and fix README badges#12

Merged
deborahjacob-botanu merged 1 commit intomainfrom
developer-deborah
Feb 6, 2026
Merged

fix: remove hardcoded URLs and fix README badges#12
deborahjacob-botanu merged 1 commit intomainfrom
developer-deborah

Conversation

@deborahjacob-botanu
Copy link
Copy Markdown
Collaborator

Security compliance improvements:

  • detector.py: Make AWS metadata endpoint configurable

    • Uses AWS SDK standard env vars (AWS_EC2_METADATA_SERVICE_ENDPOINT)
    • AWS_EC2_METADATA_DISABLED=true to disable metadata calls
    • Validate URL scheme (http/https only)
    • No custom env vars - follows AWS SDK conventions
  • README.md: Fix broken badges

    • Remove OpenSSF badge (repo not yet public)
    • Use static Python versions badge
    • Simplify PyPI version badge

Summary

Changes

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • CI / build / tooling

Testing

  • Unit tests pass (pytest)
  • Lint passes (ruff check)
  • Type check passes (mypy)

Checklist

  • My code follows the project's coding style
  • I have added SPDX headers to new files
  • I have added tests for my changes
  • I have updated documentation if needed
  • All commits are signed off (git commit -s) per the DCO

Security compliance improvements:

- detector.py: Make AWS metadata endpoint configurable
  - Uses AWS SDK standard env vars (AWS_EC2_METADATA_SERVICE_ENDPOINT)
  - AWS_EC2_METADATA_DISABLED=true to disable metadata calls
  - Validate URL scheme (http/https only)
  - No custom env vars - follows AWS SDK conventions

- README.md: Fix broken badges
  - Remove OpenSSF badge (repo not yet public)
  - Use static Python versions badge
  - Simplify PyPI version badge

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: Deborah Jacob <deborah@botanu.ai>
@deborahjacob-botanu deborahjacob-botanu merged commit aff06ce into main Feb 6, 2026
19 of 21 checks passed
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.

1 participant