Web dev proxy #172175

salemiranloye · 2025-07-15T18:13:17Z

Adding a development Proxy and dedicated Web configuration File

Flutter's current web development configuration relies on CLI arguments and does not have a development proxy. This PR adds a development proxy to flutter and a designated web_dev_config.yaml where web configuration settings are loaded from.

Issues:
#170834

Document:
HERE

Pre-launch Checklist

I read the Contributor Guide and followed the process outlined there for submitting PRs.
I read the Tree Hygiene wiki page, which explains my responsibilities.
I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
I signed the CLA.
I listed at least one issue that this PR fixes in the description above.
I updated/added relevant documentation (doc comments with ///).
I added new tests to check the change I am making, or this PR is test-exempt.
I followed the breaking change policy and added Data Driven Fixes where supported.
All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Co-authored-by: Sydney Bao [email protected]
Co-authored by: Salem Iranloye [email protected]

packages/flutter/test/material/radio_list_tile_test.dart

packages/flutter/test/widgets/tap_region_test.dart

packages/flutter_tools/lib/src/device.dart

packages/flutter/test/material/radio_list_tile_test.dart

packages/flutter_tools/lib/src/device.dart

ditman

I'm happy with how this turned out and am approving, but please don't merge until @bkonyi approves!

bkonyi

Just a couple of minor comments that wouldn't be worth blocking this PR on, but would be great to clean up if you've got the time.

LGTM and thanks for all the hard work! 🥳

bkonyi · 2025-08-07T20:42:00Z

packages/flutter_tools/lib/src/web/devfs_config.dart

+    final YamlList? proxyList = _validateType<YamlList>(value: yaml[_kProxy], fieldName: _kProxy);
+    final proxyRules = <ProxyRule>[];
+    if (proxyList != null) {
+      for (final Object? item in proxyList) {


Nit: this may be less verbose:

final proxyRules = <ProxyRule>[ ...?proxyList ?.whereType<YamlMap>() .map((e) => ProxyRule.fromYaml(e, logger)) .nonNulls, ];

bkonyi · 2025-08-07T20:44:25Z

packages/flutter_tools/lib/src/web/devfs_config.dart

+    }
+
+    final YamlList? proxyList = _validateType<YamlList>(value: yaml[_kProxy], fieldName: _kProxy);
+    final proxyRules = <ProxyRule>[];


Nit: I think this would be less verbose:

final proxyRules = <ProxyRule>[ ...?proxyList ?.whereType<YamlMap>() .map((e) => ProxyRule.fromYaml(e, logger)) .nonNulls, ];

bkonyi · 2025-08-07T20:48:21Z

packages/flutter_tools/lib/src/web/devfs_proxy.dart

+      return null;
+    } else if (target == null || target.isEmpty) {
+      effectiveLogger.printError(
+        '${ProxyRule._kLogEntryPrefix} Invalid ${ProxyRule._kTarget} for ${ProxyRule._kRegex}: $regex. ${ProxyRule._kTarget} cannot be null',


Nit: can we split these log messages across multiple lines?

fix comments

flutter/flutter@92a6bfb...3821790 2025-08-08 [email protected] Use LLDB as the default debugging method for iOS 17+ and Xcode 26+ (flutter/flutter#173443) 2025-08-08 [email protected] Roll Fuchsia Linux SDK from i4vsuEGyP8Xeb5tiy... to HclTm0V8hgSpfqmtG... (flutter/flutter#173462) 2025-08-08 [email protected] Support launching a HTTPS URL (flutter/flutter#164720) 2025-08-08 [email protected] Roll Dart SDK from c48772a79e1f to 4b7b565eb468 (1 revision) (flutter/flutter#173454) 2025-08-08 [email protected] Roll Dart SDK from ba58b96a80d7 to c48772a79e1f (3 revisions) (flutter/flutter#173451) 2025-08-07 [email protected] Web dev proxy (flutter/flutter#172175) 2025-08-07 [email protected] Roll ICU from b929596baebf to 1b2e3e8a421e (7 revisions) (flutter/flutter#173436) 2025-08-07 [email protected] [A11y] TextField prefix icon and suffix icon create a sibling node' (flutter/flutter#173312) 2025-08-07 [email protected] [Android templates] Remove jetifier usage (flutter/flutter#173431) 2025-08-07 [email protected] Remove a couple of asserts from display_list_unittest (flutter/flutter#173381) 2025-08-07 [email protected] Fix ScaffoldGeometry null scale with noAnimation FAB (flutter/flutter#172914) 2025-08-07 [email protected] Prepare for iOS debugging with lldb and devicectl (flutter/flutter#173417) 2025-08-07 [email protected] Fix `ReorderableList` proxy animation for partial drag-back (flutter/flutter#172380) 2025-08-07 [email protected] [ios26]Do not report error for Info.plist key not found (flutter/flutter#172913) 2025-08-07 [email protected] Manual roll to 3.10.0-75.1.beta (flutter/flutter#173423) 2025-08-07 [email protected] [web] add --static-assets-url argument to build web (flutter/flutter#171638) 2025-08-07 [email protected] Adds deprecation for impeller opt out on android (flutter/flutter#173375) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

woprandi · 2025-08-11T10:27:51Z

I'm very intestered so I'd like to give it a try but the basic example throw an error :

server:
  proxy: 
    - target: "https://localhost:3000"
      source: "/api/"

[ProxyRule] Invalid proxy rule in YAML: {target: https://localhost:3000, source: /api/}

The linked DOC seems outdated

mdebbar · 2025-08-11T14:02:32Z

@woprandi I'm assuming you are referring to the Google Doc, which is supposed to be a design doc and not final documentation.

Before this goes into a stable flutter release, we'll write official documentation for it.

For now, you can change source to prefix.

This also tells me we can improve on the error message to point users like yourself in the right direction 🙂

woprandi · 2025-11-06T16:33:09Z

Can we report issue about this feature ?

kevmoo · 2025-11-06T18:39:40Z

@woprandi – please! In a new issue, though.

GreatTux · 2025-11-24T14:53:27Z

Issue raised against this change here: #179014

github-actions bot added tool Affects the "flutter" command-line tool. See also t: labels. framework flutter/packages/flutter repository. See also f: labels. f: material design flutter/packages/flutter/material repository. labels Jul 15, 2025

salemiranloye force-pushed the web_dev_proxy branch 2 times, most recently from c9207cd to d801b66 Compare July 16, 2025 00:09

dkwingsmt removed framework flutter/packages/flutter repository. See also f: labels. f: material design flutter/packages/flutter/material repository. labels Jul 16, 2025

dkwingsmt requested a review from kevmoo July 16, 2025 18:18

github-actions bot added framework flutter/packages/flutter repository. See also f: labels. f: material design flutter/packages/flutter/material repository. labels Jul 16, 2025

salemiranloye force-pushed the web_dev_proxy branch from 4c9439a to c2630f8 Compare July 16, 2025 20:56

kevmoo reviewed Jul 17, 2025

View reviewed changes

packages/flutter/test/material/radio_list_tile_test.dart Show resolved Hide resolved

kevmoo reviewed Jul 17, 2025

View reviewed changes

packages/flutter/test/widgets/tap_region_test.dart Outdated Show resolved Hide resolved

kevmoo reviewed Jul 17, 2025

View reviewed changes

packages/flutter_tools/lib/src/device.dart Outdated Show resolved Hide resolved

salemiranloye force-pushed the web_dev_proxy branch 2 times, most recently from 490877e to 8404260 Compare July 17, 2025 20:18

github-actions bot removed framework flutter/packages/flutter repository. See also f: labels. f: material design flutter/packages/flutter/material repository. labels Jul 17, 2025

salemiranloye force-pushed the web_dev_proxy branch from 91628d6 to 19e3c26 Compare July 21, 2025 21:50

github-actions bot added framework flutter/packages/flutter repository. See also f: labels. f: material design flutter/packages/flutter/material repository. labels Jul 21, 2025

kevmoo reviewed Jul 21, 2025

View reviewed changes

packages/flutter/test/material/radio_list_tile_test.dart Outdated Show resolved Hide resolved

github-actions bot removed framework flutter/packages/flutter repository. See also f: labels. f: material design flutter/packages/flutter/material repository. labels Jul 21, 2025

loic-sharma reviewed Jul 22, 2025

View reviewed changes

packages/flutter_tools/lib/src/device.dart Outdated Show resolved Hide resolved

salemiranloye force-pushed the web_dev_proxy branch 5 times, most recently from 967902a to 29ce6c1 Compare July 28, 2025 17:02

ditman approved these changes Aug 7, 2025

View reviewed changes

bkonyi approved these changes Aug 7, 2025

View reviewed changes

SydneyBao and others added 2 commits August 7, 2025 14:21

fix comments

a1ea95f

Merge pull request #7 from SydneyBao/final_proxy

76988bc

fix comments

kevmoo added the autosubmit Merge PR when tree becomes green via auto submit App label Aug 7, 2025

auto-submit bot added this pull request to the merge queue Aug 7, 2025

Merged via the queue into flutter:master with commit 796c62b Aug 7, 2025
142 checks passed

flutter-dashboard bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Aug 7, 2025

This was referenced Aug 8, 2025

[web]: Support for backend proxy #51530

Closed

[web] support custom local server scenarios #67416

Open

engine-flutter-autoroll mentioned this pull request Aug 8, 2025

Roll Flutter from 92a6bfbfd6ef to ec81e2ea34c8 (14 revisions) flutter/packages#9774

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 8, 2025

796c62bd7 Web dev proxy (flutter/flutter#172175)

10dd466

engine-flutter-autoroll mentioned this pull request Aug 8, 2025

Roll Flutter from 92a6bfbfd6ef to aef15c12f410 (16 revisions) flutter/packages#9777

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 8, 2025

796c62bd7 Web dev proxy (flutter/flutter#172175)

d0ab9b2

engine-flutter-autoroll mentioned this pull request Aug 8, 2025

Roll Flutter from 92a6bfbfd6ef to 38217906e95c (17 revisions) flutter/packages#9778

Merged

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 8, 2025

796c62bd7 Web dev proxy (flutter/flutter#172175)

f342459

Piinks mentioned this pull request Aug 8, 2025

Flutter Notable Commits - 2025 #161460

Open

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 12, 2025

796c62bd7 Web dev proxy (flutter/flutter#172175)

15599d9

GreatTux mentioned this pull request Nov 24, 2025

Flutter 3.38 run command line arguments not working for web development config #179014

Open

Web dev proxy #172175

Web dev proxy #172175

Uh oh!

Conversation

salemiranloye commented Jul 15, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Pre-launch Checklist

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ditman left a comment

Choose a reason for hiding this comment

Uh oh!

bkonyi left a comment

Choose a reason for hiding this comment

Uh oh!

bkonyi Aug 7, 2025

Choose a reason for hiding this comment

Uh oh!

SydneyBao Aug 7, 2025

Choose a reason for hiding this comment

Uh oh!

bkonyi Aug 7, 2025

Choose a reason for hiding this comment

Uh oh!

SydneyBao Aug 7, 2025

Choose a reason for hiding this comment

Uh oh!

bkonyi Aug 7, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

woprandi commented Aug 11, 2025

Uh oh!

mdebbar commented Aug 11, 2025

Uh oh!

woprandi commented Nov 6, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

kevmoo commented Nov 6, 2025

Uh oh!

GreatTux commented Nov 24, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

10 participants

salemiranloye commented Jul 15, 2025 •

edited

Loading

woprandi commented Nov 6, 2025 •

edited

Loading