Skip to content

Conversation

@naveensingh
Copy link
Member

@naveensingh naveensingh commented Oct 1, 2025

Type of change(s)

  • Bug fix
  • Feature / enhancement
  • Infrastructure / tooling (CI, build, deps, tests)
  • Documentation

What changed and why

Implemented the new isContentBlocked() method from FossifyOrg/mmslib#2. This ensures that the received messages are checked for blocked keywords before they are saved to the system database.

Note: This could have been fixed easily by filtering out existing messages with blocked keywords at run time, but this way is more consistent with how messages from blocked numbers are dropped.

Tests performed

  • Receiving MMS messages with text (notification + in-app refresh)
  • Receiving MMS messages without text (notification + in-app refresh)
  • Receiving MMS messages with blocked keywords
  • Receiving MMS messages with previously blocked keywords unblocked
  • Sending MMS messages including blocked keywords
  • Sending MMS messages with or without text

I couldn't test it, but Aga-C and laubster have confirmed that this works.

Closes the following issue(s)

Checklist

  • I read the contribution guidelines.
  • I manually tested my changes on device/emulator (if applicable).
  • I updated the "Unreleased" section in CHANGELOG.md (if applicable).
  • I have self-reviewed my pull request (no typos, formatting errors, etc.).
  • All checks are passing.

@naveensingh naveensingh added the testers needed We need testers for this issue or pull request label Oct 1, 2025
@naveensingh naveensingh marked this pull request as ready for review October 5, 2025 07:13
@naveensingh naveensingh merged commit 7a291d7 into main Oct 5, 2025
7 checks passed
@naveensingh naveensingh deleted the naveensingh/fix-keyword-blocking branch October 5, 2025 07:17
@naveensingh naveensingh removed the testers needed We need testers for this issue or pull request label Oct 5, 2025
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.

Message with Blocked Keyword Got Through

2 participants