Skip to content

fix(mail): add backward-compatible redirects for mail.api.spamd.*#107

Merged
s-aga-r merged 1 commit into
frappe:developfrom
s-aga-r:fix/mail-spamd-redirects
Jun 28, 2026
Merged

fix(mail): add backward-compatible redirects for mail.api.spamd.*#107
s-aga-r merged 1 commit into
frappe:developfrom
s-aga-r:fix/mail-spamd-redirects

Conversation

@s-aga-r

@s-aga-r s-aga-r commented Jun 28, 2026

Copy link
Copy Markdown
Collaborator

Description

Adds backward-compatible whitelisted-method redirects for the standalone mail app's SpamD endpoints, now that the API lives under the combined suite app:

  • mail.api.spamd.scansuite.mail.api.spamd.scan
  • mail.api.spamd.get_spam_scoresuite.mail.api.spamd.get_spam_score

Also groups the existing redirects under section comments (Auth / Outbound / Inbound / SpamD) for readability.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@greptile-apps

greptile-apps Bot commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.
  • The new hook entries point to existing whitelisted SpamD API methods with compatible parameters.

Important Files Changed

Filename Overview
suite/hooks.py Adds two legacy SpamD whitelisted-method redirects; the target methods exist and match the expected Frappe endpoint shape.

Reviews (1): Last reviewed commit: "fix(mail): add backward-compatible redir..." | Re-trigger Greptile

@s-aga-r s-aga-r merged commit 846ce2d into frappe:develop Jun 28, 2026
5 checks passed
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