Skip to content
GitLab
Next
    • GitLab: the DevOps platform
    • Explore GitLab
    • Install GitLab
    • How GitLab compares
    • Get started
    • GitLab docs
    • GitLab Learn
  • Pricing
  • Talk to an expert
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
    Projects Groups Topics Snippets
  • Register
  • Sign in
  • GitLab GitLab
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
    • Locked files
  • Issues 51,013
    • Issues 51,013
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 1,562
    • Merge requests 1,562
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Artifacts
    • Schedules
    • Test cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Metrics
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GitLab.orgGitLab.org
  • GitLabGitLab
  • Issues
  • #329666
Closed
Open
Issue created Apr 30, 2021 by Jayson Salazar@jdsalaroDeveloper

Evaluate possible improvements to BLOCK case handling for spam verdicts.

The following discussion from !52385 (merged) should be addressed:

  • @cablett started a discussion:

    WDYT about creating an issue for whatever further iterations are planned for this? Otherwise this comment isn't very helpful since everything is already a draft 🙂


Initially, when work started on improving GitLab's spam-handling capabilities, only ALLOW, DISALLOW and CONDITIONAL_ALLOW were handled by spam_action_service.rb. However, with the introduction of https://gitlab.com/gitlab-com/gl-security/engineering-and-research/automation-team/spam/spamcheck and active usage within the product !52385 (merged), there's now a need to process all verdicts. Thus, the BLOCK case was added. At the moment, BLOCK behaves exactly as DISALLOW, however, going forward and as Spamcheck matures, we should add functionality that actually enacts a user block, upon doing so admins can get a notification, etc. This opens the door for more automations that reduce engineering time spent enacting BLOCK verdicts.

when BLOCK_USER
          # TODO: improve BLOCK_USER handling, non-existent until now
          target.spam! unless target.allow_possible_spam?
          create_spam_log(api)
Assignee
Assign to
Time tracking