feat: added the onlyoffice ee license to the ocis deployment.

[2403905]

Description

feat: added the onlyoffice ee license to the ocis deployment.

Related Issue

• Fixes <issue_link>

Motivation and Context

How Has This Been Tested?

• test environment:
• test case 1:
• test case 2:
• ...

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Technical debt
• Tests only (no source changes)

Checklist:

• Code changes
• Unit tests added
• Acceptance tests added
• Documentation ticket raised:

[update-docs]

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

[dj4oC]

I think we can archive this chmod 644 ./config/onlyoffice/license.lic with adding it to
/ocis-master/deployments/examples/ocis_full/config/onlyoffice/entrypoint-override.sh

[dj4oC]

Please add:
onlyoffice:
image: onlyoffice/documentserver-ee:9.0.4
environment:
- JWT_ENABLED=false
- REDIS_SERVER_HOST=localhost
- REDIS_SERVER_PORT=6379

[dj4oC]

.env

# --- choose edition ---
# For CE:
# ONLYOFFICE_EDITION_SUFFIX=
# For EE:
ONLYOFFICE_EDITION_SUFFIX=-ee

# Optional: pin a version (recommended)
ONLYOFFICE_TAG=latest

# Runtime env (works for both; EE typically wants these)
ONLYOFFICE_JWT_ENABLED=false
ONLYOFFICE_REDIS_HOST=localhost
ONLYOFFICE_REDIS_PORT=6379

# EE only: path to your license on the host
ONLYOFFICE_LICENSE_LOCAL=./config/onlyoffice/license.lic

docker-compose.yml

services:
  onlyoffice:
    image: onlyoffice/documentserver${ONLYOFFICE_EDITION_SUFFIX}:${ONLYOFFICE_TAG:-latest}
    environment:
      - JWT_ENABLED=${ONLYOFFICE_JWT_ENABLED:-false}
      - REDIS_SERVER_HOST=${ONLYOFFICE_REDIS_HOST:-localhost}
      - REDIS_SERVER_PORT=${ONLYOFFICE_REDIS_PORT:-6379}
    volumes:
      # Mount license; harmless for CE (file can be absent)
      - ${ONLYOFFICE_LICENSE_LOCAL:-/dev/null}:/var/www/onlyoffice/Data/license.lic:ro

[dj4oC]

I think we can archive this chmod 644 ./config/onlyoffice/license.lic with adding it to /ocis-master/deployments/examples/ocis_full/config/onlyoffice/entrypoint-override.sh

# Ensure license file has the correct permissions
if [ -f /var/www/onlyoffice/Data/license.lic ]; then
  echo "Fixing permissions on license.lic..."
  chmod 644 /var/www/onlyoffice/Data/license.lic || true
fi
``

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[Copilot]

Pull Request Overview

This PR adds support for OnlyOffice Enterprise Edition (EE) license configuration to the oCIS deployment. It introduces configurable environment variables to support both Community and Enterprise editions, with specific provisions for mounting EE license files.

Key changes:

• Added environment variables for OnlyOffice image selection and configuration
• Implemented license file mounting with proper permissions handling
• Enhanced container configuration with JWT and Redis settings for EE support

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File	Description
deployments/examples/ocis_full/web_extensions/externalsites.yml	Added volume mount for apps configuration
deployments/examples/ocis_full/onlyoffice.yml	Updated image configuration and added EE license volume mount
deployments/examples/ocis_full/config/onlyoffice/entrypoint-override.sh	Added license file permissions handling
deployments/examples/ocis_full/.env	Added OnlyOffice configuration environment variables

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Using /dev/null as a default fallback for the license file mount could cause confusion and potential issues. Consider using a more explicit approach like conditional volume mounting or a placeholder comment explaining this behavior.

Suggested change

	- ${ONLYOFFICE_LICENSE_LOCAL:-/dev/null}:/var/www/onlyoffice/Data/license.lic:ro
	# Mount the license file only if using enterprise edition. Uncomment and set ONLYOFFICE_LICENSE_LOCAL to the license file path.
	# - ${ONLYOFFICE_LICENSE_LOCAL}:/var/www/onlyoffice/Data/license.lic:ro

[2403905]

It works if a docker mounts it as - ${ONLYOFFICE_LICENSE_LOCAL:-/dev/null}:/var/www/onlyoffice/Data/license.lic:ro but in case if the ONLYOFFICE_LICENSE_LOCAL is not set the Oo will not open the document in a trial mode insted Oo said the license is expired
@dj4oC Let's keep it commented out in a onlyoffice.yml

[Copilot]

The || true construct silently ignores chmod failures which could mask legitimate permission issues. Consider logging the error or using a more specific error handling approach.

Suggested change

	chmod 644 /var/www/onlyoffice/Data/license.lic || true
	if ! chmod 644 /var/www/onlyoffice/Data/license.lic; then
	echo "Warning: Failed to set permissions on license.lic" >&2
	fi

[dj4oC]

Looks good to me but haven't tried it myself

[mmattel]

This needs a changelog entry.
I can test this earliest mid next week. Greetings from Grado/Italy 🏖️