Skip to content

Conversation

@bkonyi
Copy link
Contributor

@bkonyi bkonyi commented Aug 27, 2025

Issue Link:

What is the link to the issue this cherry-pick is addressing?

#174437

Changelog Description:

Explain this cherry pick in one line that is accessible to most Flutter developers. See best practices for examples

When running a Flutter web application in debug mode, the console is spammed with non-fatal error messages.

Impact Description:

What is the impact (ex. visual jank on Samsung phones, app crash, cannot ship an iOS app)? Does it impact development (ex. flutter doctor crashes when Android Studio is installed), or the shipping production app (the app crashes on launch)

When running a Flutter web application in debug mode, the console is spammed with DebugService: Error serving requestsError: Unsupported operation: Cannot send Null.

Workaround:

Is there a workaround for this issue?

Manually patching $PUB_CACHE/hosted/pub.dev/dwds-24.4.0+1/lib/src/services/debug_service.dart as described in #174437 (comment).

Risk:

What is the risk level of this cherry-pick?

  • Low
  • Medium
  • High

Test Coverage:

Are you confident that your fix is well-tested by automated tests?

  • Yes
  • No

Validation Steps:

What are the steps to validate that this fix works?

Run flutter run -d chrome and see that the console spam is gone.

@flutter-dashboard
Copy link

This pull request was opened from and to a release candidate branch. This should only be done as part of the official Flutter release process. If you are attempting to make a regular contribution to the Flutter project, please close this PR and follow the instructions at Tree Hygiene for detailed instructions on contributing to Flutter.

Reviewers: Use caution before merging pull requests to release branches. Ensure the proper procedure has been followed.

@bkonyi bkonyi requested a review from matanlurey August 27, 2025 21:56
@github-actions github-actions bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Aug 27, 2025
@bkonyi bkonyi changed the title [ Tool ] Roll 24.4.0+2 hotfix release of package:dwds [CP-stable][ Tool ] Roll 24.4.0+2 hotfix release of package:dwds Aug 27, 2025
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request rolls a hotfix for the dwds package, updating it to version 24.4.0+2. The change, isolated to pubspec.yaml, is intended to resolve an issue with excessive console error messages during Flutter web debugging. The update is straightforward and aligns with the PR's objective. I've reviewed the change and found no issues. The repository's style guide notes that cherry-picks like this one have typically been reviewed on the master branch already.

@matanlurey matanlurey added the autosubmit Merge PR when tree becomes green via auto submit App label Aug 27, 2025
@auto-submit auto-submit bot merged commit 6da24f0 into flutter-3.35-candidate.0 Aug 27, 2025
146 checks passed
@auto-submit auto-submit bot deleted the cp_dwds_24.4.0+2 branch August 27, 2025 22:35
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 4, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 5, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants