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

Print URI for storage objects in list-object command #544

Merged
merged 4 commits into from
Aug 30, 2023

Conversation

ZoltanFridrich
Copy link
Contributor

@ZoltanFridrich ZoltanFridrich commented Aug 28, 2023

  • makes it possible to specify an object with the URI in commands like p11-kit delete-object
  • adds URI printing for storage objects in list-objects command
  • adds test for list-objects command
  • avoid warning
  • allow arbitrary attribute lengths

@ZoltanFridrich ZoltanFridrich requested a review from ueno August 28, 2023 11:33
@ZoltanFridrich ZoltanFridrich self-assigned this Aug 28, 2023
p11-kit/delete-object.c Outdated Show resolved Hide resolved
@ZoltanFridrich ZoltanFridrich force-pushed the zfridric_devel branch 3 times, most recently from 9551bd6 to 044bf65 Compare August 29, 2023 08:41
@ZoltanFridrich ZoltanFridrich changed the title Allow all types of URIs to be used in p11-kit commands Print URI for storage objects in list-object command Aug 29, 2023
@coveralls
Copy link

coveralls commented Aug 29, 2023

Coverage Status

coverage: 68.883% (+0.3%) from 68.63% when pulling b6fae86 on ZoltanFridrich:zfridric_devel into 9ff7207 on p11-glue:master.

p11-kit/list-objects.c Outdated Show resolved Hide resolved
p11-kit/add-profile.c Outdated Show resolved Hide resolved
p11-kit/list-objects.c Outdated Show resolved Hide resolved
p11-kit/list-objects.c Outdated Show resolved Hide resolved
p11-kit/list-objects.c Outdated Show resolved Hide resolved
@ZoltanFridrich ZoltanFridrich force-pushed the zfridric_devel branch 2 times, most recently from a3ff192 to b6fae86 Compare August 30, 2023 08:56
@ZoltanFridrich ZoltanFridrich requested a review from ueno August 30, 2023 08:58
p11-kit/list-objects.c Show resolved Hide resolved
p11-kit/list-objects.c Show resolved Hide resolved
@ZoltanFridrich ZoltanFridrich requested a review from ueno August 30, 2023 10:20
@ZoltanFridrich ZoltanFridrich merged commit a3549f8 into p11-glue:master Aug 30, 2023
@ueno ueno added this to the 0.25.1 milestone Oct 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants