cfug / dio

A powerful HTTP client for Dart and Flutter, which supports global settings, Interceptors, FormData, aborting and canceling a request, files uploading and downloading, requests timeout, custom adapters, etc.
https://dio.pub
MIT License
12.49k stars 1.51k forks source link

Dio.request always timeout on android device when receiving data #1932

Closed lyf571321556 closed 1 year ago

lyf571321556 commented 1 year ago

Package

dio

Version

5.3.2

Output of flutter doctor -v

[✓] Flutter (Channel stable, 3.0.4, on macOS 13.0 22A380 darwin-arm, locale
    zh-Hans-CN)
    • Flutter version 3.0.4 at /Users/ones/Library/flutter
    • Upstream repository git@github.com:flutter/flutter.git
    • Framework revision 85684f9300 (1 year, 1 month ago), 2022-06-30 13:22:47
      -0700
    • Engine revision 6ba2af10bb
    • Dart version 2.17.5
    • DevTools version 2.12.2
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[✗] Android toolchain - develop for Android devices
    • Android SDK at /Users/ones/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.

Dart Version

2.17.5

Steps to Reproduce

import 'package:dio/dio.dart';
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        // This is the theme of your application.
        //
        // Try running your application with "flutter run". You'll see the
        // application has a blue toolbar. Then, without quitting the app, try
        // changing the primarySwatch below to Colors.green and then invoke
        // "hot reload" (press "r" in the console where you ran "flutter run",
        // or simply save your changes to "hot reload" in a Flutter IDE).
        // Notice that the counter didn't reset back to zero; the application
        // is not restarted.
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key, required this.title}) : super(key: key);

  // This widget is the home page of your application. It is stateful, meaning
  // that it has a State object (defined below) that contains fields that affect
  // how it looks.

  // This class is the configuration for the state. It holds the values (in this
  // case the title) provided by the parent (in this case the App widget) and
  // used by the build method of the State. Fields in a Widget subclass are
  // always marked "final".

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  Future<void> _incrementCounter() async {
    try {
      final dio = Dio();
      dio.options
        ..baseUrl = 'https://ones-liuyanfeng.myones.net/api/project/'
        ..connectTimeout = const Duration(seconds: 5)
        ..receiveTimeout = const Duration(seconds: 10);
      final params = {"email": "marsdev+01@ones.ai", "password": "Test1234"};
      // final result = await dio.post("auth/v2/login", data: params);
      // final result = await dio.request("auth/v2/login",
      //     data: params,
      //     options: Options(
      //       method: 'POST',
      //     ));
      final result = await dio.request("auth/login_support",
          options: Options(
            method: 'GET',
          ));
      print(result.toString());
    } catch (e) {
      print(e);
    }
    setState(() {
      // This call to setState tells the Flutter framework that something has
      // changed in this State, which causes it to rerun the build method below
      // so that the display can reflect the updated values. If we changed
      // _counter without calling setState(), then the build method would not be
      // called again, and so nothing would appear to happen.
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    // This method is rerun every time setState is called, for instance as done
    // by the _incrementCounter method above.
    //
    // The Flutter framework has been optimized to make rerunning build methods
    // fast, so that you can just rebuild anything that needs updating rather
    // than having to individually change instances of widgets.
    return Scaffold(
      appBar: AppBar(
        // Here we take the value from the MyHomePage object that was created by
        // the App.build method, and use it to set our appbar title.
        title: Text(widget.title),
      ),
      body: Center(
        // Center is a layout widget. It takes a single child and positions it
        // in the middle of the parent.
        child: Column(
          // Column is also a layout widget. It takes a list of children and
          // arranges them vertically. By default, it sizes itself to fit its
          // children horizontally, and tries to be as tall as its parent.
          //
          // Invoke "debug painting" (press "p" in the console, choose the
          // "Toggle Debug Paint" action from the Flutter Inspector in Android
          // Studio, or the "Toggle Debug Paint" command in Visual Studio Code)
          // to see the wireframe for each widget.
          //
          // Column has various properties to control how it sizes itself and
          // how it positions its children. Here we use mainAxisAlignment to
          // center the children vertically; the main axis here is the vertical
          // axis because Columns are vertical (the cross axis would be
          // horizontal).
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const Text(
              'You have pushed the button this many times:',
            ),
            Text(
              '$_counter',
              style: Theme.of(context).textTheme.headline4,
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    );
  }
}

Expected Result

image

image

Actual Result

DioException [receive timeout]: The request took longer than 0:00:10.000000 to receive data. It was aborted.

lyf571321556 commented 1 year ago

it works well when Downgrading to version 4.0.6.

AlexV525 commented 1 year ago

image

I can see the first request timed out but the next requests are successful. This might result in the host taking a while to seek through the DNS server. You can try to extend the connect timeout to 15~30 seconds which is a common value in production apps.

Closing as not reproducible. If you have other regular ways to always reproduce the issue, please provider them.

lyf571321556 commented 1 year ago

image

I can see the first request timed out but the next requests are successful. This might result in the host taking a while to seek through the DNS server. You can try to extend the connect timeout to 15~30 seconds which is a common value in production apps.

Closing as not reproducible. If you have other regular ways to always reproduce the issue, please provider them.

@AlexV525 ,I found that when the parameter 'data' passed to ‘dio.get()’ is not of type ‘String’, this issue can be reproduced. try { final dio = Dio(); dio.options ..baseUrl = 'https://ones-liuyanfeng.myones.net/api/project/' ..connectTimeout = const Duration(seconds: 15) ..receiveTimeout = const Duration(seconds: 30); final result = await dio.get("auth/login_support", data: {}, options: Options( method: 'GET', )); print(result.toString()); } catch (e) { print(e); }