Skip to content

feat(attributes): Add integrity policy attributes - #686

Closed
GemTails wants to merge 2 commits into
getsentry:mainfrom
GemLunaMarine:main
Closed

GemTails wants to merge 2 commits into
getsentry:mainfrom
GemLunaMarine:main

Conversation

@GemTails

@GemTails GemTails commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Description

getsentry/relay#6465
Currently Relay doesn't support Integrity policy. This is required for the above getsentry/relay Pull request to work.

PR Checklist

  • I have run yarn test and verified that the tests pass.
  • I have run yarn generate to generate and format code and docs.

If an attribute was added:

  • The attribute is in a namespace (e.g. nextjs.function_id, not function_id)
  • I have used the correct value for apply_scrubbing (i.e. manual or auto. Use never only for values that should never be scrubbed such as IDs)

If an attribute was deprecated: (NO DEPRECIATION)

@loewenheim loewenheim changed the title Added integration of Integrity Policy feat(attributes): Add integrity policy attributes Oct 7, 2026

@loewenheim loewenheim left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I took the liberty of changing the title to be in line with our conventions, other than that it looks good to me :)

@loewenheim

Copy link
Copy Markdown
Contributor

@GemTails please merge the current main into your branch and run yarn generate again.

@GemTails
GemTails force-pushed the main branch 4 times, most recently from 53feff1 to e7ad960 Compare October 8, 2026 10:49
@GemTails

GemTails commented Oct 8, 2026

Copy link
Copy Markdown
Contributor Author

Finished updating the commit

@loewenheim

loewenheim commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

My apologies, but it looks like you have to run yarn generate again. If you'd rather not deal with this I can also take over this PR.

@GemLunaMarine

Copy link
Copy Markdown

Sure. I don't mind, as long as it works

@loewenheim

Copy link
Copy Markdown
Contributor

See #691.

@loewenheim loewenheim closed this Oct 9, 2026
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.

3 participants