Skip to content

Conversation

@b-long
Copy link
Owner

@b-long b-long commented Aug 11, 2025

✨🪩✨ New and Improved! ✨🪩✨

This pull request rewrites this library in "pure python" (of course, using the cryptography library means that Rust is used under the hood for crypto).

In this PR, we introduce a simplified, modernized toolchain. In particular, the CI pipeline and local development is improved using using the uv tool. The most important changes are grouped below.

Build and Test Improvements:

Cleanup and Redundancy Removal:

  • Deletes obsolete workflows related to gopy tooling

These changes significantly simplify the CI/CD process, improve maintainability, and align the workflows with modern Python tooling and best practices.

TODO

  • Replace uses of getattr and kwargs
  • Remove commented code (see _get_token_from_client_credentials)
  • Remove hardcoded realm_name = "opentdf"
  • Update README
  • Reintroduce cibuildwheel components for multi-arch distribution, and upload built wheels to PyPI

TODO (nice to have)

b-long added 30 commits July 7, 2025 18:22
- When run with the proper .env file: 7 failed, 142 passed, 2 skipped, 1 warning
- Critical naming fix
- Update .proto files
- Add script to update .proto files
- Ditch HTTP impl
- Improve manifest and encrypt test
- Python CLI decrypt now works correctly with TDF files created by otdfctl
@b-long b-long marked this pull request as ready for review August 11, 2025 01:13
@b-long b-long merged commit 02f51cd into develop Aug 11, 2025
6 checks passed
@b-long b-long deleted the chore/rewrite-in-python branch August 23, 2025 15:42
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