fix: updating some apps via Marketplace ignores permissions

[d-gubert]

Proposed changes (including videos or screenshots)

When an app that requests custom permissions would be updated via the Marketplace, the endpoint handler would not assign a valid value to the permissionsGranted field sent to the apps-engine as a parameter. This would cause the checksum to be calculated without taking into account any permissions, but when validating the installation, the permissionsGranted field would still contain data - this would cause the app to be marked as invalid_installation_disabled.

Issue(s)

CONN-387

Steps to test or reproduce

Further comments

[changeset-bot]

🦋 Changeset detected

Latest commit: 2eb5bd5

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 36 packages

Name	Type
@rocket.chat/apps-engine	Patch
@rocket.chat/meteor	Patch
@rocket.chat/apps	Patch
@rocket.chat/core-services	Patch
@rocket.chat/core-typings	Patch
@rocket.chat/fuselage-ui-kit	Patch
@rocket.chat/rest-typings	Patch
@rocket.chat/ddp-streamer	Patch
@rocket.chat/presence	Patch
rocketchat-services	Patch
@rocket.chat/uikit-playground	Patch
@rocket.chat/api-client	Patch
@rocket.chat/cron	Patch
@rocket.chat/ddp-client	Patch
@rocket.chat/freeswitch	Patch
@rocket.chat/gazzodown	Patch
@rocket.chat/livechat	Patch
@rocket.chat/model-typings	Patch
@rocket.chat/ui-contexts	Patch
@rocket.chat/account-service	Patch
@rocket.chat/authorization-service	Patch
@rocket.chat/omnichannel-transcript	Patch
@rocket.chat/presence-service	Patch
@rocket.chat/queue-worker	Patch
@rocket.chat/stream-hub-service	Patch
@rocket.chat/license	Patch
@rocket.chat/omnichannel-services	Patch
@rocket.chat/pdf-worker	Patch
@rocket.chat/models	Patch
@rocket.chat/network-broker	Patch
@rocket.chat/ui-avatar	Patch
@rocket.chat/ui-client	Patch
@rocket.chat/ui-video-conf	Patch
@rocket.chat/ui-voip	Patch
@rocket.chat/web-ui-registration	Patch
@rocket.chat/instance-status	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[dionisio-bot]

Looks like this PR is ready to merge! 🎉
If you have any trouble, please check the PR guidelines

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 59.43%. Comparing base (9e84ebc) to head (2eb5bd5).
Report is 1 commits behind head on develop.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop   #35170   +/-   ##
========================================
  Coverage    59.43%   59.43%           
========================================
  Files         2825     2825           
  Lines        68155    68155           
  Branches     15123    15123           
========================================
  Hits         40505    40505           
  Misses       24992    24992           
  Partials      2658     2658           

Flag	Coverage Δ
unit	75.68% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.