Unverified Commit d6a25ae6 authored by Ben Konyi's avatar Ben Konyi Committed by GitHub

Temporarily disable Dart Development Service in flutter_tools (#62508)

Devtools doesn't currently support connections to DDS. Disable DDS
temporarily while a solution is worked on. See https://github.com/flutter/flutter/issues/62507
parent 22bf19ce
...@@ -15,6 +15,9 @@ import 'logger.dart'; ...@@ -15,6 +15,9 @@ import 'logger.dart';
class DartDevelopmentService { class DartDevelopmentService {
DartDevelopmentService({@required this.logger}); DartDevelopmentService({@required this.logger});
// TODO(bkonyi): enable once VM service can handle SSE forwarding for
// Devtools (https://github.com/flutter/flutter/issues/62507)
static const bool ddsDisabled = true;
final Logger logger; final Logger logger;
dds.DartDevelopmentService _ddsInstance; dds.DartDevelopmentService _ddsInstance;
...@@ -22,6 +25,13 @@ class DartDevelopmentService { ...@@ -22,6 +25,13 @@ class DartDevelopmentService {
Uri observatoryUri, Uri observatoryUri,
bool ipv6, bool ipv6,
) async { ) async {
if (ddsDisabled) {
logger.printTrace(
'DDS is currently disabled due to '
'https://github.com/flutter/flutter/issues/62507'
);
return;
}
final Uri ddsUri = Uri( final Uri ddsUri = Uri(
scheme: 'http', scheme: 'http',
host: (ipv6 ? host: (ipv6 ?
......
...@@ -6,6 +6,7 @@ import 'dart:async'; ...@@ -6,6 +6,7 @@ import 'dart:async';
import 'dart:io'; // ignore: dart_io_import import 'dart:io'; // ignore: dart_io_import
import 'package:file/file.dart'; import 'package:file/file.dart';
import 'package:flutter_tools/src/base/dds.dart';
import 'package:flutter_tools/src/base/file_system.dart'; import 'package:flutter_tools/src/base/file_system.dart';
import 'package:matcher/matcher.dart'; import 'package:matcher/matcher.dart';
import 'package:vm_service/vm_service.dart'; import 'package:vm_service/vm_service.dart';
...@@ -46,7 +47,7 @@ void main() { ...@@ -46,7 +47,7 @@ void main() {
for (final Protocol protocol in protocolList.protocols) { for (final Protocol protocol in protocolList.protocols) {
expect(protocol.protocolName, anyOf('VM Service', 'DDS')); expect(protocol.protocolName, anyOf('VM Service', 'DDS'));
} }
}); }, skip: DartDevelopmentService.ddsDisabled);
test('flutterVersion can be called', () async { test('flutterVersion can be called', () async {
final Response response = final Response response =
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment