-
-
Notifications
You must be signed in to change notification settings - Fork 195
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
Wrong response type for getAllCertificates call #72
Conversation
This call has a response that corresponds to ProfileDevicesLinkagesResponse, not ProfileResponse. According to the documentation it should be a CertificatesResponse (see https://developer.apple.com/documentation/appstoreconnectapi/list_all_certificates_in_a_profile), but that's not correct either, since the attributes are missing from the certificates in the data array. An alternative option could be to make the attributes optional in certificate and use CertificatesResponse here. For now I propose to abuse ProfileDevicesLinkagesResponse or else create a new response type specifically for this. What do you think?
@ristkof to me it makes the most sense to create a new response for this that matches the outcome. Would you mind doing that instead? FYI; I'm working on fixing CI today, so you don't have to worry about that! And thanks for your contribution 💪 |
AppStoreConnect_Swift_SDK.framework: Coverage: 81.63
Generated by 🚫 Danger Swift against 296e844 |
@AvdLee Changed as requested. |
Congratulations! 🎉 This was released as part of Release 1.0.1 🚀 |
This call has a response that corresponds to ProfileDevicesLinkagesResponse, not ProfileResponse. According to the documentation it should be a CertificatesResponse (see https://developer.apple.com/documentation/appstoreconnectapi/list_all_certificates_in_a_profile), but that's not correct either, since the attributes are missing from the certificates in the data array.
An alternative option could be to make the attributes optional in certificate and use CertificatesResponse here.
For now I propose to abuse ProfileDevicesLinkagesResponse or else create a new response type specifically for this.
What do you think?
Screenshot of Charles-dump attached (with some private bits cut out).