client icon indicating copy to clipboard operation
client copied to clipboard

Team identifier missing in release builds

Open erikjv opened this issue 4 years ago • 4 comments

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

erikjv avatar Nov 11 '21 13:11 erikjv

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

TheOneRing avatar Nov 11 '21 14:11 TheOneRing

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

Done

erikjv avatar Nov 17 '21 13:11 erikjv

Enough to fix this for 2.10?

dschmidt avatar Nov 17 '21 13:11 dschmidt

Enough to fix this for 2.10?

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

erikjv avatar Nov 17 '21 13:11 erikjv