search

dart-lang / sdk

The Dart SDK, including the VM, JS and Wasm compilers, analysis, core libraries, and more.
https://dart.dev
BSD 3-Clause "New" or "Revised" License
10.25k stars 1.58k forks source link

Flutter and VM behaving differently on data reception #34591

Closed shamblett closed 6 years ago

shamblett commented 6 years ago

I have this issue raised on one of my packages. Basically its saying when data is received from, in this case an MQTT broker, Dart on the VM is OK no matter what the data size, but flutter seems to have a problem in that it raises exceptions when processing the data.

If there is a fix for this then I guess its in the flutter runtime or maybe in typed_data, i..e at a lower level than a package.

matanlurey commented 6 years ago

I would continue to raise this issue https://github.com/shamblett/mqtt_client/issues/34 or on the Flutter repository (https://github.com/flutter/flutter/issues/new), there isn't much actionable here.

zoechi commented 6 years ago

Moved to https://github.com/flutter/flutter/issues/22657

zoechi commented 6 years ago

@mraleph should we close https://github.com/flutter/flutter/issues/22657 ? I don't see the benefit of tracking it twice.

mraleph commented 6 years ago

There is absolutely no reason to file this over at Flutter because both execution and networking are handled by the Dart VM. So it's going to be exactly the same people looking at the problem no matter where you file the bug.

@shamblett we can look into the issue if you have a standalone reproduction e.g. a Flutter project that can be cloned and run on a device or in the emulator to reproduce the problem.

Without reproduction we can't really do much, unfortunately.

@zanderso Zach, do you by chance know if dart:io has any issues with networking on Android that can lead to message truncation?

@zoechi please close that one.

shamblett commented 6 years ago

I don't have a flutter environment but the guys reporting this do, as you can see from the issue @cbenhagen is using flutter on android, @Smartauto and @josvos also have environments, I think one is emulated and they all see the same thing, suggest you ask one of those for an environment set up.

mraleph commented 6 years ago

@shamblett from my reading of the comments on the issue it actually looks like the issue is resolved (at least for @josvos, and seems for @Smartauto too but they are just not testing it correctly). So I am going to go ahead and mark this issue as resolved too.