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.

Sign up for GitHub

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

Jump to bottom

[TECH DEBT] Replace libnacl with pynacl #64372

Closed
dwoz opened this issue May 26, 2023 · 2 comments
Closed

[TECH DEBT] Replace libnacl with pynacl #64372

dwoz opened this issue May 26, 2023 · 2 comments
Assignees
@dwoz @dmurphy18
Labels
Bug broken, incorrect, or confusing behavior tech-debt
Milestone
Sulfur v3006.2

Comments

@dwoz
Copy link
Contributor

dwoz commented May 26, 2023

libnacl has been plagued with issues for some time now. We are no longer actively maintaining libnacl. An alternative, pynacl is being actively maintained and has all the bits we need for salt. Migrate from libnacl to pynacl. All of the changes needed should be in salt.utils.nacl.
@dwoz dwoz added Bug broken, incorrect, or confusing behavior tech-debt labels May 26, 2023
@dwoz
Copy link
Contributor Author

dwoz commented May 26, 2023

See also: #64342

@dwoz dwoz added this to the Sulfur v3006.2 milestone May 26, 2023
@dwoz dwoz mentioned this issue May 26, 2023
Closed
9 tasks
@dwoz dwoz assigned dmurphy18 and dwoz and unassigned dmurphy18 May 26, 2023
@dwoz dwoz mentioned this issue May 31, 2023
Closed
3 tasks
@dmurphy18 dmurphy18 mentioned this issue Jun 2, 2023
Merged
3 tasks
@dmurphy18
Copy link
Contributor

Closing this since PR #64415 is merged

PaulChristophel added a commit to PaulChristophel/docker-salt that referenced this issue Sep 7, 2023
@PaulChristophel
Per saltstack/salt#64372 switch to pynacl
cf1e82c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dwoz dwoz

@dmurphy18 dmurphy18

Labels
Bug broken, incorrect, or confusing behavior tech-debt
Projects
None yet
Milestone
Sulfur v3006.2
Development

No branches or pull requests
2 participants
@dwoz @dmurphy18