Skip to content

Commit

Permalink
Enable automatic underscore replace for primary-bundle-id
Browse files Browse the repository at this point in the history
  • Loading branch information
MichalMMac committed Jun 17, 2019
1 parent 5af2ae3 commit 81fec0e
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -272,7 +272,7 @@ Keys/values of the `notarization_info` dictionary:
| username | String | Yes | Login email adress of your developer Apple ID |
| password | String | Yes | 2FA app specific password. For information about the password and saving it to the login keychain see the web page [Customizing the Notarization Workflow](https://developer.apple.com/documentation/security/notarizing_your_app_before_distribution/customizing_the_notarization_workflow) |
| asc_provider | String | No | Only needed when a user account is associated with multiple providers |
| primary_bundle_id | String | No | Defaults to `identifier`. `primary_bundle_id` is useful when `identifier` contains characters such as '_' Apple notary service does not like |
| primary_bundle_id | String | No | Defaults to `identifier`. Whether specified or not underscore characters are always automatically converted to hyphens since Apple notary service does not like underscores |
| staple_timeout | Integer | No | See paragraph bellow |

**About accessing password in keychain**
Expand Down
2 changes: 1 addition & 1 deletion munkipkg
Original file line number Diff line number Diff line change
Expand Up @@ -675,7 +675,7 @@ def upload_to_notary(build_info, options):
'--primary-bundle-id',
build_info['notarization_info'].get(
'primary_bundle_id', build_info['identifier']
),
).replace('_', '-'),
'--username',

This comment has been minimized.

Copy link
@gregneagle

gregneagle Jun 17, 2019

Contributor

This quickly gets unreadable. I suggest a "get_primary_bundle_id" function here.

This comment has been minimized.

Copy link
@MichalMMac

MichalMMac Jun 18, 2019

Author Contributor

See fixup 8d449ba

build_info['notarization_info']['username'],
'--password',
Expand Down

0 comments on commit 81fec0e

Please sign in to comment.