Skip to content

chore(issues): Remove obsolete HPKP/Expect-CT/Expect-Staple tests - #121028

Merged
mrduncan merged 1 commit into
masterfrom
mrduncan/remove-obsolete-tests
Aug 3, 2026
Merged

chore(issues): Remove obsolete HPKP/Expect-CT/Expect-Staple tests#121028
mrduncan merged 1 commit into
masterfrom
mrduncan/remove-obsolete-tests

Conversation

@mrduncan

Copy link
Copy Markdown
Member

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Jul 31, 2026
@mrduncan
mrduncan marked this pull request as ready for review August 1, 2026 00:29
@mrduncan
mrduncan requested a review from a team as a code owner August 1, 2026 00:29
@mrduncan
mrduncan merged commit 95a1ec8 into master Aug 3, 2026
63 checks passed
@mrduncan
mrduncan deleted the mrduncan/remove-obsolete-tests branch August 3, 2026 17:37
mrduncan added a commit to getsentry/relay that referenced this pull request Aug 4, 2026
Follow-up to 9fbaceb, which stopped accepting these reports at ingest but
deliberately kept the protocol surface for a coordinated removal. This does that
removal: the EventType variants, the hpkp / expectct / expectstaple fields on the
Event schema, and the now-unreachable interface types and their raw parsers.

Ingestion behavior is unchanged -- reports are already rejected at the endpoint
with an `invalid` outcome. What changes is that an event which an older upstream
Relay already classified as one of these types is now forwarded as a `default`
event, so it counts against the error quota rather than the security quota.

getsentry/sentry#121028 removed the Sentry-side tests that exercised these types
through Relay's normalizer, so this no longer breaks Sentry CI. Removing the dead
Sentry source (eventtypes, interfaces, grouping strategies) is a follow-up that
must land after Sentry bumps its sentry-relay dependency.

Refs getsentry/sentry#119638
Refs getsentry/sentry#121028
Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants