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

fixed: app-extension code sign and auto signature/provisioning profile pickup #293

Merged
merged 5 commits into from May 19, 2018

Conversation

Projects
None yet
2 participants
@dkimitsa
Copy link
Contributor

dkimitsa commented May 16, 2018

this PR contains two related to each-other parts:

  1. it fixes auto signature/provisioning profile pickup as it was broken when was trying to pickup signature -- it resulted in random first one that matches iPhone Developer|iOS Development and often it doesn't match profile
  2. fix for code-sign of app-extension: it is required to embed own extension provisioning profile and entitlements
  3. provisioning profile is auto picked up but config was extended to allow custom profile to be specified for extension:
<appExtensions>
    <extension profile="3AED05A9-5F1F-4120-9276-11980B9C88EE">OneSignalNotificationServiceExtension</extension>
</appExtensions>

dkimitsa added some commits May 16, 2018

* fixed: auto signature/provisioning profile pick/up:
Old code was broken due following: if signature was set to 'Auto' it was picking any `iPhone Developer|iOS Development` one without any respect to provisioning profile. This caused following use cases broken:
1. `auto signature` and `manual profile`: randomly picked signature will not match profile (in case you are not lucky and have lot of signatures/profiles);
2. `auto signature` and `auto profile` will not work as well as signature is random;

Fix provides following:
1. `auto signature` and `manual profile`: picks only signatures that matches profile;
2. `auto signature` and `auto profile`: first set of profiles that match bundle id is fetched. This set is being intersected with list of identities to find a profile and signature pair that matches each other.
* fixed: auto signature/provisioning profile pick/up (continued to IO…
…STarget)

* app-extension fixed to add own provision profile and embed entitlements into signature
@dkimitsa

This comment has been minimized.

Copy link
Contributor Author

dkimitsa commented May 16, 2018

lets wait for confirmation from @sofroma

@dkimitsa

This comment has been minimized.

Copy link
Contributor Author

dkimitsa commented May 16, 2018

@dkimitsa

This comment has been minimized.

Copy link
Contributor Author

dkimitsa commented May 17, 2018

3rd commit copies support files needed for stickers app ext, check gitter message for details.

@dkimitsa

This comment has been minimized.

Copy link
Contributor Author

dkimitsa commented May 18, 2018

as per @sofroma comment it is working now and can be reviewed

@florianf florianf merged commit d5029b4 into MobiVM:master May 19, 2018

@dkimitsa dkimitsa deleted the dkimitsa:appext_appstore_fix branch May 23, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.