APN_CERTIFICATE_PASSPHRASE was never being used because the passphrase would always have a value (the one provided by the user or the empty string).
When using an unencrypted pem (not password protected) the command line would complain unless you provided some 4+ byte value password. This was because when no password was provided it would use the empty string and complain that the password was not long enough. See [comment] (https://github.com/nomad/houston/issues/146#issuecomment-359273095)
This pull request allows the password to be nil to fix the above issues.
dankimio
commented
6 years ago
This fixes two bugs in the
apncommand line tool.APN_CERTIFICATE_PASSPHRASEwas never being used because the passphrase would always have a value (the one provided by the user or the empty string).This pull request allows the password to be nil to fix the above issues.