Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

More reverts for insecure http policy since the Dart implementation is reverted #25623

Closed
wants to merge 2 commits into from

Conversation

xster
Copy link
Member

@xster xster commented Apr 16, 2021

Reverting due to flutter/flutter#72723.

Need to be re-implemented at some point for flutter/flutter#54448.

Reverting this part because it uses APIs (via vm-entrypoint) to be reverted in https://dart-review.googlesource.com/c/sdk/+/195485. Removed an additional read against the Android SDK because it would be slow and doesn't feed into anything anymore.

@chinmaygarde
Copy link
Member

Can we rebase this to make the presubmits pass and land this please?

@xster xster force-pushed the more-http-revert branch from 5f13df9 to 6caca84 Compare April 28, 2021 16:25
@chinmaygarde
Copy link
Member

No sure the presubmit failures are flakes.

@xster
Copy link
Member Author

xster commented Apr 29, 2021

Ya, I think something's properly broken. It didn't reproduce locally. I'll still have to dig in a bit when I got some time.

@chinmaygarde
Copy link
Member

Added the WIP tag in the meantime.

@chinmaygarde chinmaygarde added the Work in progress (WIP) Not ready (yet) for review! label May 6, 2021
@jmagman
Copy link
Member

jmagman commented May 18, 2021

Before this lands make sure it doesn't regress flutter/flutter#77132 again. It was easy to test locally with the Info.plist change reported in the issue.

@Hixie
Copy link
Contributor

Hixie commented Nov 2, 2021

Since this has apparently been abandoned I'm going to close it; please let me know if we need to track something.

@Hixie Hixie closed this Nov 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants