Skip to content

Conversation

@yipfram
Copy link
Contributor

@yipfram yipfram commented Oct 24, 2025

Changes

  • Unsend rebranded to Usesend
  • Pinned image version
  • changed logo and deleting old one
  • Set github creds as mandatory

@ShadowArcanist
Copy link
Member

Hey, there is already an PR (#6606) for this exact same change and it got everything working but need minor changes.

We will wait some more time for that PR author to implement the requested change, if they didn't respond then we can consider this PR.

@yipfram
Copy link
Contributor Author

yipfram commented Oct 24, 2025

Hey, there is already an PR (#6606) for this exact same change and it got everything working but need minor changes.

We will wait some more time for that PR author to implement the requested change, if they didn't respond then we can consider this PR.

Sorry i did not check... Will do next time !

I updated the docs, adding SMTP documentation also

@Cinzya Cinzya added the UI/UX label Oct 26, 2025
@yipfram
Copy link
Contributor Author

yipfram commented Oct 28, 2025

? @Cinzya
Why ui ux tag for this PR?

@ShadowArcanist
Copy link
Member

? @Cinzya Why ui ux tag for this PR?

She probably thought its a UI related PR after reading the first part of the Title "Remove unused svg" xD

Next time please start the title like fix(service):

@ShadowArcanist ShadowArcanist added ⚙️ Service Issues requesting or PRs adding/fixing service templates. and removed UI/UX labels Oct 28, 2025
@ShadowArcanist ShadowArcanist self-assigned this Oct 28, 2025
@github-actions
Copy link
Contributor

Hi @yipfram! 👋

It appears to us that you are either adding a new service or making changes to an existing one.
We kindly ask you to also review and update the Coolify Documentation to include this new service or it's new configuration needs.
This will help ensure that our documentation remains accurate and up-to-date for all users.

Coolify Docs Repository: https://github.com/coollabsio/coolify-docs
How to Contribute a new Service to the Docs: https://coolify.io/docs/get-started/contribute/service#adding-a-new-service-template-to-the-coolify-documentation

@yipfram yipfram changed the title Remove unused unsend SVG and add usesend configuration file fix(service): add usesend configuration file and Remove unused unsend SVG Oct 29, 2025
@yipfram
Copy link
Contributor Author

yipfram commented Oct 29, 2025

Fixed title

@yipfram
Copy link
Contributor Author

yipfram commented Oct 29, 2025

BTW, if this PR (#6606) is not changed and we end up using mine, i'll probably change the Cert system, which can make this template click and deploy...

For now you have to make changes in the server proxy and compose file... That's too complicated if we can just attach let'sencrypt... But Idk if this works for everyone (idk how those work...)

@ShadowArcanist
Copy link
Member

Hey @yipfram thanks for the PR, for now I am putting this PR on Hold since that other PR author responded + implemented some changes (waiting to implement few more).

The changes on this PR for the usesend.yaml is same as the other PR, so let's wait and see if that PR author addresses my questions + provides a way to test the changes - if they don't then we can merge this one (I may forget this PR so feel free to ping me here or in discord after 2 weeks of this comment)

@ShadowArcanist ShadowArcanist added the ⏸️ Hold Issues or PRs that are on hold while awaiting planning or decisions from the core team. label Nov 13, 2025
@yipfram
Copy link
Contributor Author

yipfram commented Nov 13, 2025

Ok, in the meantime I'll check the other cert system and see how this can work

@yipfram
Copy link
Contributor Author

yipfram commented Nov 14, 2025

Could we make this work? @ShadowArcanist

USESEND_API_CERT_PATH: "${TLS_CERT}"
USESEND_API_KEY_PATH: "${TLS_KEY}"
ports:
- "25:25" # Standard SMTP
- "587:587" # SMTP with STARTTLS
- "2587:2587" # Alternative SMTP port
- "465:465" # SMTP over SSL/TLS
- "2465:2465" # Alternative SMTPS port
volumes:
- type: bind
source: /etc/letsencrypt
target: /certs

Like Letsencrypt

@ShadowArcanist
Copy link
Member

Could we make this work? @ShadowArcanist

USESEND_API_CERT_PATH: "${TLS_CERT}"
USESEND_API_KEY_PATH: "${TLS_KEY}"
ports:
- "25:25" # Standard SMTP
- "587:587" # SMTP with STARTTLS
- "2587:2587" # Alternative SMTP port
- "465:465" # SMTP over SSL/TLS
- "2465:2465" # Alternative SMTPS port
volumes:
- type: bind
source: /etc/letsencrypt
target: /certs

Like Letsencrypt

Sadly I have little to no experience with email related stuff so I can't suggest/recommend anything here without testing + understanding stuffs on my own

@yipfram
Copy link
Contributor Author

yipfram commented Nov 14, 2025

Basically it works, the only question is how you collect the certs for the smtp server. I don't think it's related to email stuff

@ShadowArcanist
Copy link
Member

Basically it works, the only question is how you collect the certs for the smtp server. I don't think it's related to email stuff

I think you can simply map the /data/coolify/proxy/acme.json (proxy stores the certs there), if the user is using custom certs the you can map /data/coolify/proxy/certs

@yipfram
Copy link
Contributor Author

yipfram commented Nov 14, 2025

Acme can work in the place of cert and key file?
Like as place of the cert path?

Sorry I'm on mobile now, I can't give an example snippet

@ShadowArcanist
Copy link
Member

Acme can work in the place of cert and key file? Like as place of the cert path?

Sorry I'm on mobile now, I can't give an example snippet

I just checked it on my dev instance and we can't use acme.json if we need a cert and key files T-T, so we may need something like traefik cert dumper

@yipfram
Copy link
Contributor Author

yipfram commented Nov 14, 2025

Ok, any way we could attach it to this docker compose? (Not the proxy one)

@ShadowArcanist
Copy link
Member

Ok, any way we could attach it to this docker compose? (Not the proxy one)

I think it's only needed for the proxy to work, if they are not using the usesend proxy then no need for certs (I maybe wrong here)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⏸️ Hold Issues or PRs that are on hold while awaiting planning or decisions from the core team. ⚙️ Service Issues requesting or PRs adding/fixing service templates.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants