Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow symbols in breadcrumbs #561

Closed
wants to merge 1 commit into from

Conversation

directionless
Copy link
Contributor

Validate Symbol as an allowed type to breadcrumbs. I can't tell if this will have break something else, I can't trace everywhere it's used.

Fixes #539

Validate Symbol as an allowed type to breadcrumbs. I can't tell if this will have break something else, I can't trace everywhere it's used. 

Fixes bugsnag#539
@abigailbramble
Copy link

@directionless - thank you for this. We will have a look at your pull request.

@abigailbramble abigailbramble added the needs discussion Requires internal analysis/discussion label Aug 22, 2019
@tomlongridge
Copy link
Contributor

Fixing under #563

@abigailbramble abigailbramble added scheduled Work is starting on this feature/bug released This feature/bug fix has been released and removed needs discussion Requires internal analysis/discussion scheduled Work is starting on this feature/bug labels Aug 27, 2019
@abigailbramble
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released This feature/bug fix has been released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Breadcrumb metadata validation fails when calling Rails.cache.fetch
3 participants