Skip to content

Add Mailcow Mail Server support#227

Merged
mayanayza merged 1 commit intoscanopy:devfrom
fthomys:feat/mailcow
Dec 10, 2025
Merged

Add Mailcow Mail Server support#227
mayanayza merged 1 commit intoscanopy:devfrom
fthomys:feat/mailcow

Conversation

@fthomys
Copy link
Contributor

@fthomys fthomys commented Dec 1, 2025

Adds basic mailcow detection support

@mayanayza
Copy link
Collaborator

mayanayza commented Dec 1, 2025

Thanks for the contribution! Please make this PR to dev not main :)

@fthomys fthomys changed the base branch from main to dev December 1, 2025 22:33
@gitguardian
Copy link

gitguardian bot commented Dec 1, 2025

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
While these secrets were previously flagged, we no longer have a reference to the
specific commits where they were detected. Once a secret has been leaked into a git
repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@fthomys
Copy link
Contributor Author

fthomys commented Dec 1, 2025

Thanks for the contribution! Please merge this to dev :)

sorry, my bad. changed the base to dev

@fthomys
Copy link
Contributor Author

fthomys commented Dec 2, 2025

is this okay? Or should i reopen a new pr?

@mayanayza
Copy link
Collaborator

hey @fthomys, there are a few files unrelated to the change which made it into the PR - can you re-push without those files?

specifically - close-stale.yml and README.md

Once that's all set, I'll go ahead and merge!

@fthomys fthomys force-pushed the feat/mailcow branch 2 times, most recently from 3d25b7d to 42efa42 Compare December 8, 2025 14:02
@fthomys
Copy link
Contributor Author

fthomys commented Dec 8, 2025

Done!

@mayanayza
Copy link
Collaborator

@fthomys looks like the formatting check failed, can you run make format and make lint on your code and push it again?

@fthomys
Copy link
Contributor Author

fthomys commented Dec 9, 2025

Now its done!

@mayanayza
Copy link
Collaborator

Looks like one of the tests failed due the the logo URL not being from one of the allowed sources; can you switch it to https://cdn.jsdelivr.net/gh/homarr-labs/dashboard-icons/svg/mailcow.svg?

Thanks for your patience with this!

@fthomys
Copy link
Contributor Author

fthomys commented Dec 10, 2025

Looks like one of the tests failed due the the logo URL not being from one of the allowed sources; can you switch it to https://cdn.jsdelivr.net/gh/homarr-labs/dashboard-icons/svg/mailcow.svg?

Thanks for your patience with this!

Done, and no problem, was my bad.
Should be fully resolved now

@mayanayza mayanayza merged commit 1b3bb62 into scanopy:dev Dec 10, 2025
3 checks passed
mayanayza added a commit that referenced this pull request Feb 24, 2026
Add Mailcow Mail Server support
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.

2 participants