Skip to content

Rename InboundFilters integration to EventFilters #15431

Open
@Lms24

Description

@Lms24

Description

If we get a chance to change this, I propose to rename the inboundFiltersIntegration because its name doesn't make a lot of sense. Open for other suggestions but logically, outboundFiltersIntegration makes way more sense.

After a bit of reflection, I think outboundFilters only makes sense if you knew the previous name, so I propose something like EventFilters for a fresh perspective :)

Reasons why I think "InboundFilters" is a confusing name:

  • Nothing is "inbound" in the SDK world in the sense that we would filter incoming events. On the contrary, this integration filters out outbound events the SDK would send to Sentry.
  • We have a product feature called "Inbound Filters" which users sometimes confuse or mix up with the SDK integration.
  • The integration name does not reflect the purpose well. Granted, eventFilters isn't perfect either but at least it suggests that events (errors and transactions) get filtered out by this integration.

This integration is likely used a lot so we should probably leave it deprecated for a major or two before we actually remove it.

Steps:

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions