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

Fix base64 encode #7558

Draft
wants to merge 4 commits into
base: development
Choose a base branch
from
Draft

Fix base64 encode #7558

wants to merge 4 commits into from

Conversation

mpg
Copy link
Contributor

@mpg mpg commented May 5, 2023

Description

Resolve #7550

PR checklist

Please tick as appropriate and edit the reasons (e.g.: "backport: not needed because this is a new feature")

  • changelog provided, or not required
  • backport done, or not required
  • tests provided, or not required

mpg added 4 commits May 5, 2023 13:06
@mpg
Use set_step() in test loop
e90ac48 
Signed-off-by: Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com>
@mpg
Rewrite PEM write length check for readability
c680ebe 
Signed-off-by: Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com>
@mpg
Use modern test macros in base64 test suite
32d05e0 
Also wrap a few long lines while at it.

Signed-off-by: Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com>
@mpg
WIP: Use exact-size buffers in base64 test suite
af143bf 
Also avoid unnecessary copies while at it.

Signed-off-by: Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com>
@mpg mpg added DO-NOT-MERGE needs-ci Needs to pass CI tests labels May 5, 2023
@mpg mpg self-assigned this May 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mpg mpg

Labels
DO-NOT-MERGE needs-ci Needs to pass CI tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Complete fix for pem write with certain lengths
1 participant
@mpg