[cronet_http] Upgrade jni to 0.9.2 and publish 1.2.1

[HosseinYousefi]

Fix our users' issue described here: dart-lang/native#1119.

@brianquinlan Please manually publish as cronet currently needs Flutter and it won't work with the autopublish CI.

[brianquinlan]

The tests timed-out so I restarted them. I have one more Dart 3.4-releated change to land and then I will publish this.

[github-actions]

Package publishing

Package	Version	Status	Publish tag (post-merge)
package:http_profile	0.1.0	ready to publish	http_profile-v0.1.0
package:cronet_http	1.2.1-wip	WIP (no publish necessary)
package:cupertino_http	1.4.1-wip	WIP (no publish necessary)
package:http	1.2.2-wip	WIP (no publish necessary)
package:web_socket	0.1.3	already published at pub.dev

Documentation at https://github.com/dart-lang/ecosystem/wiki/Publishing-automation.

[brianquinlan]

The emulator isn't even starting before the tests time out. Hopefully this is a temporary github glitch.

[techouse]

I encountered the error described in dart-lang/native#1119 when I upgraded AGP to v8.4.0. It works fine with AGP v8.3.2.

[HosseinYousefi]

I encountered the error described in dart-lang/native#1119 when I upgraded AGP to v8.4.0. It works fine with AGP v8.3.2.

but the upgrade to minSdk 21 fixes it on AGP v8.4.0, right?

[techouse]

Not on the app level. My

android.defaultConfig.minSdkVersion 24

still fails with the same error.

It has most likely something to do with the jni plugin.

[brianquinlan]

I think that the CI is broken. Working on fixing it.

[HosseinYousefi]

Not on the app level. My

android.defaultConfig.minSdkVersion 24

still fails with the same error.

It has most likely something to do with the jni plugin.

I meant after the fix to jni. You can override the jni dependency to use 0.9.2 and try.

[HosseinYousefi]

I think that the CI is broken. Working on fixing it.

Thanks! Merge this whenever you see fit then.

[techouse]

I meant after the fix to jni. You can override the jni dependency to use 0.9.2 and try.

You mean just use this PR instead of v1.2.0? Sure, gimme a few minutes.

[HosseinYousefi]

I meant after the fix to jni. You can override the jni dependency to use 0.9.2 and try.

You mean just use this PR instead of v1.2.0? Sure, gimme a few minutes.

Yeah, you can do that too,

Here's the dependency!

cronet_http:
  git:
    url: https://github.com/dart-lang/http
    ref: jni-0.9.2
    path: pkgs/cronet_http

[techouse]

Yep, compiles ok using

• Flutter: v3.19.6
• Kotlin: v1.9.24
• AGP: 8.4.0

Running OK on Android 14 (Pixel 6) with this config

dependency_overrides:
  cronet_http:
    git:
      url: https://github.com/dart-lang/http.git
      ref: jni-0.9.2
      path: pkgs/cronet_http

and android/app/build.gradle

dependencies {
  implementation(platform("org.jetbrains.kotlin:kotlin-bom:1.9.24"))
}

(I'm not 100% sure this is still needed.)

It only complains about

/Users/techouse/.pub-cache/hosted/pub.dev/jni-0.9.2/android/src/main/java/com/github/dart_lang/jni/JniPlugin.java uses or overrides a deprecated API.

So I guess you're good to go 🚀

[HosseinYousefi]

It only complains about

/Users/techouse/.pub-cache/hosted/pub.dev/jni-0.9.2/android/src/main/java/com/github/dart_lang/jni/JniPlugin.java uses or overrides a deprecated API.

Yeah, that's not a problem. Although, I thought I have silenced this warning here: https://github.com/dart-lang/native/blob/main/pkgs/jni/android/src/main/java/com/github/dart_lang/jni/JniPlugin.java#L22 🤔