Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor: Restructure Library, Introduce pyspiffe-tls Module, and Simplify Interface #146

Merged
merged 1 commit into from
Mar 24, 2024

Conversation

maxlambrecht
Copy link
Collaborator

@maxlambrecht maxlambrecht commented Mar 24, 2024

PR Description:

This PR introduces a comprehensive overhaul of the py-spiffe library, aimed at improving usability, maintainability, and providing a foundation for future expansion. Here are the key changes and additions:

  • Core Module Restructuring: I've restructured the pyspiffe core module to make it more intuitive and easier to use.

  • New pyspiffe-tls Module: Addition of the pyspiffe-tls module, currently in the initial stages of development. This module aims to supplement the core functionalities with TLS utilities, enabling seamless integration of SPIFFE identities into Python applications’ TLS workflows. The pyspiffe-tls module currently doesn't have any functionality, it's only the main folders and configuration files.

  • Simplified Dependencies and Tooling: streamlined dependencies and tools.

  • Introduction of Type Re-exports: reducing the need for extensive import paths and simplifying codebases that utilize the library.

  • Codebase Cleanup: removed unnecessary files and outdated documentation.

  • Updated Documentation: The README for the pyspiffe core module now includes usage instructions and examples.

@maxlambrecht maxlambrecht merged commit 09da14f into HewlettPackard:main Mar 24, 2024
6 checks passed
@maxlambrecht maxlambrecht deleted the library-modules branch March 24, 2024 00:24
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