Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix secrets usage and files cleanup #4

Merged
merged 1 commit into from
Apr 25, 2024
Merged

Fix secrets usage and files cleanup #4

merged 1 commit into from
Apr 25, 2024

Conversation

chinmaya-n
Copy link
Contributor

No description provided.

{{- if .Values.maildev.config.smtp.incoming.username }}
- name: MAILDEV_INCOMING_USER
value: {{ .Values.maildev.config.smtp.incoming.username | quote }}
{{- end }}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So does this mean if auth is required, but no username is provided, you still need to provide the smtp-incoming-password?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. That is the assumption I'm making b'se that is how the original Chart was. I assumed there is probably a default username the application assumes if nothing is provided. I'm not sure about the application behavior, so, I didn't deviate much from the original chart.

nodePort: {{ .Values.services.smtp.nodePort }}
{{ end }}
{{- if (eq .Values.services.smtp.type "NodePort") }}
{{- if .Values.services.smtp.nodePort }}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why reconstruct this way? Is the and operator not working ? Just curious.

Copy link
Contributor Author

@chinmaya-n chinmaya-n Apr 25, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't confidently understand the AND operation. Documentation Here.

and computes the Boolean AND of its arguments, returning the first false argument it encounters, or the last argument.

@alluen alluen merged commit 68affc4 into OIT-SET:main Apr 25, 2024
@chinmaya-n chinmaya-n deleted the cs/fix-secret-usage branch April 25, 2024 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants