Skip to content

Remove upper limit for max_breadcrumbs option - #953

Merged
cleptric merged 3 commits into
getsentry:masterfrom
FeBe95:patch-1
Sep 23, 2025
Merged

Remove upper limit for max_breadcrumbs option#953
cleptric merged 3 commits into
getsentry:masterfrom
FeBe95:patch-1

Conversation

@FeBe95

@FeBe95 FeBe95 commented Sep 12, 2025

Copy link
Copy Markdown
Contributor

My pull request getsentry/sentry-php#1890 was just merged.

Now, I've implemented the same change for the Sentry Symfony repo, so that the configuration validation matches again.

Note

There's not yet a new release for sentry/sentry, but the change should be included in the upcoming 4.15.3.

@Litarnus Litarnus 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.

Nice! Thanks for your contribution

@cleptric cleptric left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We need to bump the base SDK to 4.16.0 in the composer.json.

@cleptric cleptric left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nvm, we did this in #960.

@cleptric
cleptric enabled auto-merge (squash) September 23, 2025 10:37
@cleptric
cleptric merged commit ce1c283 into getsentry:master Sep 23, 2025
39 of 40 checks passed
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