include smtp credentials except for authentication mode `none` - Githubissues

owncloud / ocis-charts

:chart_with_upwards_trend: Helm Charts for ownCloud's OCIS

https://owncloud.dev/ocis/deployment/kubernetes/

Apache License 2.0

46 stars 27 forks source link

include smtp credentials except for authentication mode `none` #589

Closed wkloucek closed 4 months ago

wkloucek commented 4 months ago

Description

actually provide smtp credentials when they are needed, compare to the actual secret reference description:

https://github.com/owncloud/ocis-charts/blob/f4280747eb43935d07103ed585ffcc58c537b952/charts/ocis/values.yaml#L596-L599

Related Issue

Fixes https://github.com/owncloud/ocis-charts/issues/585

Motivation and Context

How Has This Been Tested?

not yet

Screenshots (if appropriate):

Types of changes

[x] 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:

[x] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation generated (make docs) and committed
[ ] Documentation ticket raised:
[ ] Documentation PR created: