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

Recursion during print() #2669

Open
oscarshaitan opened this issue Feb 6, 2025 · 7 comments
Open

Recursion during print() #2669

oscarshaitan opened this issue Feb 6, 2025 · 7 comments
Assignees

Comments

@oscarshaitan
Copy link

Platform

Flutter Web

Obfuscation

Enabled

Debug Info

Enabled

Doctor

[✓] Flutter (Channel stable, 3.27.4, on macOS 15.0 24A335 darwin-arm64 (Rosetta), locale en-GB)
• Flutter version 3.27.4 on channel stable at /Users/oscartigreros/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision d8a9f9a52e (6 days ago), 2025-01-31 16:07:18 -0500
• Engine revision 82bd5b7209
• Dart version 3.6.2
• DevTools version 2.40.3

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
• Android SDK at /Users/oscartigreros/Library/Android/sdk
• Platform android-34, build-tools 34.0.0
• Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)
• All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 16.0)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 16A242d
• CocoaPods version 1.15.2

[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.1)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

[✓] IntelliJ IDEA Community Edition (version 2023.1.7)
• IntelliJ at /Applications/IntelliJ IDEA CE.app
• Flutter plugin version 78.2.1
• Dart plugin version 231.9414.10

[✓] Connected device (4 available)
• sdk gphone64 arm64 (mobile) • emulator-5554 • android-arm64 • Android 14 (API 34) (emulator)
• macOS (desktop) • macos • darwin-arm64 • macOS 15.0 24A335 darwin-arm64 (Rosetta)
• Mac Designed for iPad (desktop) • mac-designed-for-ipad • darwin • macOS 15.0 24A335 darwin-arm64 (Rosetta)
• Chrome (web) • chrome • web-javascript • Google Chrome 133.0.6943.53

[✓] Network resources
• All expected network resources are available.

• No issues found!

Version

8.13.0

Steps to Reproduce

I use
logger: ^2.5.0

since I updated
sentry_flutter from 8.9.0 to 8.12.0 (tested on 8.13.0 also present)

the logger throws
Recursion during print() call.Abort adding print() call as Breadcrumb.

I tested with other loggers same experience

Expected Result

Image

Actual Result

Image

Are you willing to submit a PR?

None

@buenaflor
Copy link
Contributor

Hey this is a warning that means we are aborting adding the print statement as a breadcrumb.

but we'll have a look, cc @denrase

@oscarshaitan
Copy link
Author

According to what I spot
https://github.com/getsentry/sentry-dart/pull/2088/files

was the PR that introduce the SentryRunZonedGuarded and the bug.

@getsantry getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 3 Feb 7, 2025
@oscarshaitan
Copy link
Author

Update: not sure if is worthy
but the log works fine if I put the initialization inside a runZonedGuarded

@oscarshaitan
Copy link
Author

oscarshaitan commented Feb 7, 2025

still think is a bug but at least that solves my issue. I closed cuz I have nothing more to add

@oscarshaitan oscarshaitan reopened this Feb 10, 2025
@denrase
Copy link
Collaborator

denrase commented Feb 11, 2025

Hey @oscarshaitan, which issue are you still experiencing after the fix?

@oscarshaitan
Copy link
Author

if I use Sentry outside runZonedGuarded the logger error persist

@denrase
Copy link
Collaborator

denrase commented Feb 11, 2025

@oscarshaitan This should have been fixed in 8.13.0, I'll check again why this is still occurring.

@denrase denrase moved this from Needs Investigation to In Investigation in Mobile & Cross Platform SDK Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Status: In Investigation
Development

No branches or pull requests

3 participants