Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Use &str for key names instead of String #81

Closed
hug-dev opened this issue Apr 29, 2021 · 0 comments · Fixed by #86
Closed

Use &str for key names instead of String #81

hug-dev opened this issue Apr 29, 2021 · 0 comments · Fixed by #86
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@hug-dev
Copy link
Member

hug-dev commented Apr 29, 2021

In all methods of the BasicClient that need a key. See here for reference.
@hug-dev hug-dev added enhancement New feature or request good first issue Good for newcomers labels Apr 29, 2021
MattDavis00 added a commit to MattDavis00/parsec-client-rust that referenced this issue Jul 8, 2021
@MattDavis00
Changed to use &str for key_name parameters.
679ad53 
Used to have to call key_name.clone() when passing it as an argument.
Now it follows the pattern of psa_generate_key(&key_name, key_attrs)

Issue parallaxsecond#81

Signed-off-by: Matt Davis <matt.davis@arm.com>
@MattDavis00 MattDavis00 mentioned this issue Jul 8, 2021
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Changed to use &str for key_name parameters. MattDavis00/parsec-client-rust
1 participant
@hug-dev