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

Empty Entitlements when building for dist-adhoc or dist-appstore #8403

Closed
wants to merge 1 commit into from

Conversation

kopiro
Copy link
Contributor

@kopiro kopiro commented Sep 20, 2016

JIRA: https://jira.appcelerator.org/browse/AC-4482

CRITICAL bug: when building for App store or Adhoc, an empty Entitlements file is created.
This behaviour cause the automatic rejection from Apple when you're using push notifications or other features that need a valid entry in that file:

Missing Push Notification Entitlement - Your app includes an API for Apple's Push Notification service, but the aps-environment entitlement is missing from the app's signature. 
To resolve this, make sure your App ID is enabled for push notification in the Provisioning Portal. Then, sign your app with a distribution provisioning profile that includes the aps-environment entitlement. This will create the correct signature, and you can resubmit your app. 
See "Provisioning and Development" in the Local and Push Notification Programming Guide for more information. If your app does not use the Apple Push Notification service, no action is required. You may remove the API from future submissions to stop this warning. 
If you use a third-party framework, you may need to contact the developer for information on removing the API.

@hansemannn
Copy link
Collaborator

Good catch! But the change is missing a use-case where a enterprise-certificate is used. @cb1kenobi is currently setting up a PR that handles both. It will go into 5.5.1 for sure!

@cb1kenobi
Copy link
Contributor

Thanks for the PR! I've got a fix that encompasses this fix here: #8416.

@cb1kenobi cb1kenobi closed this Sep 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants