Skip to content

[FIX] sentry: Fix KeyError 'request_bodies' on DEFAULT_OPTIONS - #3105

Open
hnavarro-kernet wants to merge 1 commit into
OCA:16.0from
hnavarro-kernet:hodei-server-tools-sentry-patch-3
Open

[FIX] sentry: Fix KeyError 'request_bodies' on DEFAULT_OPTIONS#3105
hnavarro-kernet wants to merge 1 commit into
OCA:16.0from
hnavarro-kernet:hodei-server-tools-sentry-patch-3

Conversation

@hnavarro-kernet

Copy link
Copy Markdown

See original migration for reference: getsentry/sentry-python@2b1d1cc

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @barsi, @versada, @naglis, @moylop260, @fernandahf,
some modules you are maintaining are being modified, check this out!

@MiquelRForgeFlow

Copy link
Copy Markdown
Contributor

@hnavarro-kernet could you rebase?

@hnavarro-kernet
hnavarro-kernet force-pushed the hodei-server-tools-sentry-patch-3 branch from d4ebe51 to 03af76f Compare February 2, 2025 23:54
@hnavarro-kernet

Copy link
Copy Markdown
Author

Should be rebased now @MiquelRForgeFlow

@hnavarro-kernet
hnavarro-kernet force-pushed the hodei-server-tools-sentry-patch-3 branch from 03af76f to bfb9d40 Compare February 3, 2025 11:08
@hnavarro-kernet

Copy link
Copy Markdown
Author

Pre-commit successful, waiting for tests/test now.

@moylop260

Copy link
Copy Markdown
Contributor

the test is red, could you help us, please?

@hnavarro-kernet
hnavarro-kernet force-pushed the hodei-server-tools-sentry-patch-3 branch from bfb9d40 to 331c57a Compare February 13, 2025 07:16
@hnavarro-kernet

Copy link
Copy Markdown
Author

All 4 PRs depend on each other I guess:

So maybe merge them into one?

Maybe backport from this one that should have fixed all the issues?

If a maintainer with approval rights want to tell me what to do I'd be happy to, if not, this has been on prod since the PR was opened so I'm not going to fight the tests.

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.

4 participants