[spaceship] Support individual API key

[manicmaniac]

Checklist

• I've run bundle exec rspec from the root directory to see all new and existing tests pass
• I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
• I see several green ci/circleci builds in the "All checks have passed" section of my PR (connect CircleCI to GitHub if not)
• I've read the Contribution Guidelines
• I've updated the documentation if necessary.
• I've added or updated relevant unit tests.

Motivation and Context

Resolves #21959

Description

The main difference between a team API key and an individual API key is that the latter does not have issuer_id.

So I made issuer_id optional and if it does not exist, fastlane treats the key as an individual API key.

Since the JWT token payload of an individual API key is a bit different from a team API key, I changed the logic to generate token with following the official document.

https://developer.apple.com/documentation/appstoreconnectapi/generating_tokens_for_api_requests#4313913

Testing Steps

Sorry, I don't have a good idea to test whether if the generated JWT payload is valid or not, without exposing my individual API key.

However I tested it by writing a small lane like the following.

# fastlane/Fastfile
lane :test do
  app_store_connect_api_key(
    key_id: ENV.fetch('INDIVIDUAL_API_KEY_ID'),
    key_content: ENV.fetch('INDIVIDUAL_API_KEY_CONTENT_BASE64'),
    is_key_content_base64: true,
    in_house: false,
    duration: 500
  )
  puts Spaceship::ConnectAPI::App.all
end

and confirmed that it works well in my private repo.

[lucgrabowski]

Thank you for your contribution :)) 💪
The code looks good to me 🔥, I'll add a comment with an idea for a test in spec.

[lucgrabowski]

Thanks for contribution and adding tests :)

[anasyarn]

Please merge this as soon as possible, waiting for this change for a long!!!

[anasyarn]

Please merge @manicmaniac

[manicmaniac]

@anasyarn Sorry I don't have the permission to do it, just wait for the fastlane team's reaction. Or you can use my fork until then.

[max-ott]

@anasyarn Did you test this? Would love to merge this, but as it's a rather "big" change, we need to ensure enough people from the community tested it.

[anasyarn]

No i haven’t tested it.

…

On Fri 3 May 2024 at 3:05 PM, Max Ott ***@***.***> wrote: @anasyarn <https://github.com/anasyarn> Did you test this? Would love to merge this, but as it's a rather "big" change, we need to ensure enough people from the community tested it. — Reply to this email directly, view it on GitHub <#21987 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AJ7U4QCCJFRWYJ56SDPU55DZAN4SLAVCNFSM6AAAAABGSTOEJCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJSHA4DENZVGQ> . You are receiving this because you were mentioned.Message ID: ***@***.***>