Skip to content

build(Makefile): update gdata to gdatavaas in apps list#126

Draft
printminion-co wants to merge 2 commits into
mainfrom
mk/dev/configure-gdatavaas-app
Draft

build(Makefile): update gdata to gdatavaas in apps list#126
printminion-co wants to merge 2 commits into
mainfrom
mk/dev/configure-gdatavaas-app

Conversation

@printminion-co
Copy link
Copy Markdown
Contributor

This change updates the application list in the Makefile to replace 'gdata_antivirus' with 'gdatavaas', reflecting the new application configuration.

This change updates the application list in the Makefile to replace
'gdata_antivirus' with 'gdatavaas', reflecting the new application
configuration.

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Add configure_gdatavaas_app() function to configure.sh that enables
and configures the GDATA VaaS antivirus app via OCC commands.

https://github.com/GDATASoftwareAG/nextcloud-gdata-antivirus?tab=readme-ov-file#configuring-via-the-command-line

Features:
- Conditional enablement via APP_GDATAVAAS_ENABLED flag
- Supports both ResourceOwnerPassword and ClientCredentials auth methods
- Configures all GDATA VaaS settings with sensible defaults
- Handles sensitive credentials with --sensitive flag
- Validates required environment variables per auth method
- Uses set_app_config_typed() for proper type handling

Configuration includes:
- Authentication (username/password or clientId/clientSecret)
- VaaS and token endpoint URLs (defaults to staging)
- Quarantine folder settings
- Auto-scan and file tagging options
- Scan filters (allowlist/blocklist)
- Email notifications for infected files
- Scan limits (max file size, timeout)
- Caching and hash lookup options

All environment variables follow APP_GDATAVAAS_* naming convention
as documented in JIRA ticket for helm chart integration.

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
@printminion-co printminion-co marked this pull request as draft March 11, 2026 10:26
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

Successfully merging this pull request may close these issues.

1 participant