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

Only strip pkcs8 header for Private Keys (Fixes Issue #85) #86

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

LowAmmo
Copy link
Contributor

@LowAmmo LowAmmo commented Feb 5, 2025

  • Also address build warning about an unused variable

Description

Fix unit test failure due to stripping the header of all pkcs8 keys - instead of JUST the private keys.

Motivation and Context

Seeing a unit test failure in Xcode 15 & 16, when targeting Mac or iPhone Simulator.

How Has This Been Tested?

Verified successful unit tests

Checklist:

  • If applicable, I have updated the documentation accordingly.
  • If applicable, I have added tests to cover my changes.

* Also address build warning about an unused variable
Copy link

sonarqubecloud bot commented Feb 5, 2025

@LowAmmo
Copy link
Contributor Author

LowAmmo commented Feb 5, 2025

@dannys42 - Will need you to pull this in before I can do the releasing and then the tagging of 1.0.203 for cocoapods

@LowAmmo
Copy link
Contributor Author

LowAmmo commented Feb 5, 2025

@dannys42 - Not sure how to get the Travis CI Build to start working... seems like it's looking for an install script in the https://github.com/Kitura/Package-Builder repo that isn't there???

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.

1 participant