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

Team identifier missing in release builds #9209

Open
erikjv opened this issue Nov 11, 2021 · 4 comments
Open

Team identifier missing in release builds #9209

erikjv opened this issue Nov 11, 2021 · 4 comments
Assignees
Milestone

Comments

@erikjv
Copy link
Collaborator

erikjv commented Nov 11, 2021

When doing a release build, the CMake variable SOCKETAPI_TEAM_IDENTIFIER_PREFIX is not set to the team identifier. The team identifier looks like 4AP2STM4H5 , and the variable should include an extra period, eg: 4AP2STM4H5.

To see the team identifier used to build the app bundle, you can use:

codesign -dvvvv owncloud.app/Contents/MacOS/owncloud | grep TeamIdentifier
TeamIdentifier=4AP2STM4H5

This team-id is already propagated to the FinderSyncExt, and should show up in the com.apple.security.application-groups array as 4AP2STM4H5.com.owncloud.desktopclient This can be verified by running:

codesign -d --entitlements :- owncloud.app/Contents/PlugIns/FinderSyncExt.appex/Contents/MacOS/FinderSyncExt
@TheOneRing
Copy link
Member

Shouldn't we add the extra period to the code that needs it and not to the variable itself?

@erikjv
Copy link
Collaborator Author

erikjv commented Nov 17, 2021

Shouldn't we add the extra period to the code that needs it and not to the variable itself?

Done

@dschmidt
Copy link
Member

Enough to fix this for 2.10?

@erikjv
Copy link
Collaborator Author

erikjv commented Nov 17, 2021

Enough to fix this for 2.10?

If the cmake invocation of the CI/release-build-thingy is done, then yes.

@TheOneRing TheOneRing modified the milestones: 3.0, 4.0 May 18, 2022
@TheOneRing TheOneRing modified the milestones: 4.0, 5.0 Jan 26, 2023
@TheOneRing TheOneRing modified the milestones: Desktop 6.0, Inbox Sep 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants