Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed changes
Fixes #71
Removing conditions on import 'package:http/io_client.dart'; and import 'dart:io'; does not causes crash on flutter web.
Some tests with --platform chrome|firefox still failing but errors are caused by other stuff.
I did some tests running my changes in a real flutter web application and no error was thrown other than previous mentioned (only for web). See Further comments section for details.
This fix will allow users to use this library on flutter web although with limited functionalities.
Type of changes
Checklist
Further comments
After running my changes on real flutter web application an XMLHttpRequest error was thrown, but I solved that by configuring properly my backend CORS configuration. This is the same error that occurs during tests with --platform chrome.
Unfortunately, verify: false for SSL validity verify still not working on web. This require some large implementation (maybe also in http library, I didn't analyzed that so much). A new issue will have to be opened after merge.