Skip to content

Add a hex wrapper #70

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

Merged
merged 8 commits into from
Jun 25, 2025
Merged

Add a hex wrapper #70

merged 8 commits into from
Jun 25, 2025

Conversation

scr-oath
Copy link
Collaborator

@scr-oath scr-oath commented Jun 23, 2025

@dborovcanin - this adds hex support #69

  1. Sorted plugin Loaders
  2. Implemented hex with unit tests/readme

I did NOT make the change (to crypto aes stuff) here, but I also recommend updating your code in #69 maybe with _hex suffixes if you find them useful, but mainly to just convert strings to []byte directly.

@scr-oath scr-oath requested a review from vadv June 23, 2025 23:21
@scr-oath
Copy link
Collaborator Author

scr-oath commented Jun 24, 2025

@dborovcanin - see #71 for how the non-hex could be added. Note - I totally agree that, for testing, hex can be useful… I guess I could code up the actual use case, if slightly contrived, which is to read from file.

@scr-oath
Copy link
Collaborator Author

@vadv - This PR is discrete - while motivated by #69 - this only adds the hex wrapper, which others might find useful on its own.

@scr-oath scr-oath merged commit 1aed397 into master Jun 25, 2025
6 checks passed
@scr-oath scr-oath deleted the hex branch June 25, 2025 19:55
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