[url_launcher] Set broadcast reciever visability as required by target api 34 #3973

reidbaker · 2023-05-12T16:42:15Z

Update the version of androidx used.
Explicitly set Exported on previously exported broadcast reciever
126460

This change is a no-op in behavior. I am open to the argument that instead we make this configurable.

https://developer.android.com/guide/components/broadcasts#context-registered-receivers

Pre-launch Checklist

flutter-dashboard · 2023-05-12T16:54:40Z

It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat (don't just cc him here, he won't see it! He's on Discord!).

If you are not sure if you need tests, consider this rule of thumb: the purpose of a test is to make sure someone doesn't accidentally revert the fix. Ask yourself, is there anything in your PR that you feel it is important we not accidentally revert back to how it was before your fix?

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

stuartmorgan-g · 2023-05-12T17:06:54Z

...l_launcher_android/android/src/main/java/io/flutter/plugins/urllauncher/WebViewActivity.java

@@ -144,6 +145,11 @@ public void onCreate(@Nullable Bundle savedInstanceState) {

    // Register receiver that may finish this Activity.
    registerReceiver(broadcastReceiver, closeIntentFilter);


Do we still need this version?

No, removed.

stuartmorgan-g

LGTM! One nit I missed before.

stuartmorgan-g · 2023-05-12T18:43:58Z

packages/url_launcher/url_launcher_android/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 6.0.33
+
+* Explicitly set if reciever for close should be exported.


Nit: sets
(Per repo changelog style guide)

packages/url_launcher/url_launcher_android/CHANGELOG.md

Hixie · 2023-05-12T19:43:39Z

test-exempt: SDK34 is only in beta and is not yet in our testing matrix; this will be tested once SDK34 is part of our normal testing regime.

stuartmorgan-g · 2023-05-13T17:33:30Z

FYI this caused OOB breakage for us when it was published: https://github.com/flutter/packages/runs/13447851341

e: [...]/annotation.kotlin_module: Module was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.8.0, expected version is 1.6.0.
e: [...]/kotlin-stdlib-jdk7.kotlin_module: Module was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.8.0, expected version is 1.6.0.
e: [...]/kotlin-stdlib-jdk8.kotlin_module: Module was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.8.0, expected version is 1.6.0.

It has a very clear error message from flutter tooling though, so I don't think we need to revert or retract; affected clients will have an obvious, actionable way to fix it:

┌─ Flutter Fix ──────────────────────────────────────────────────────────────────────────────┐
│ [!] Your project requires a newer version of the Kotlin Gradle plugin.                     │
│ Find the latest version on https://kotlinlang.org/docs/releases.html#release-details, then │
│ update /Users/stuartmorgan/src/packages/packages/go_router/example/android/build.gradle:   │
│ ext.kotlin_version = '<latest-version>'                                                    │
└────────────────────────────────────────────────────────────────────────────────────────────┘

I can repro locally; it appears that having an explicit kotlin_version of 1.6.x in an app conflicts with this; interestingly I only have to update to 1.7.x (I used 1.7.10, which is what current flutter create uses) to make it work, even though the trigger is using 1.8.x

I'll update all our examples that have a Kotlin version to 1.7.10, but we may need to do the same in flutter/flutter/

#3973 caused an out-of-band failure after publishing, because an example that uses `url_launcher` had a too-old Kotlin version set. This is not something we consider client-breaking because `flutter` prodives a very clear error message with a straightforward and actionable fix step (update the app's Kotlin version), so the fix for the breakage is just to update our own examples. Since increasingly we're likely to hit problems where modern version of dependencies don't work with old version of Kotlin, this adds repo-wide CI enforcement that examples are set to a minimum version (matching the current `flutter/flutter` template; we can increase this over time as we feel it's useful to do so).

…by target api 34 (flutter/packages#3973)

flutter/packages@6328cfb...6608b60 2023-05-15 [email protected] [ci] Remove unnecessary Linux libraries (flutter/packages#3975) 2023-05-15 [email protected] [ci] Enforce a minimum Kotlin version in examples (flutter/packages#3979) 2023-05-15 [email protected] [ci] Disable Windows repo tool tests (flutter/packages#3980) 2023-05-12 [email protected] [ci] Add LUCI repo tool tests (flutter/packages#3964) 2023-05-12 [email protected] [url_launcher] Set broadcast reciever visability as required by target api 34 (flutter/packages#3973) 2023-05-12 [email protected] Update build.gradle with a comment explaining version (flutter/packages#3972) 2023-05-12 [email protected] Roll Flutter from 8c5a1ea to a76dbe4 (12 revisions) (flutter/packages#3958) 2023-05-12 [email protected] [image_picker] Fix Android lints (flutter/packages#3887) 2023-05-12 [email protected] [various] Update minimum Flutter version to 3.3 (flutter/packages#3967) 2023-05-12 [email protected] [tool] Target specific Android unit tests (flutter/packages#3955) 2023-05-12 [email protected] [camerax] Add a dependency on kotlin-bom to align versions of kotlin-stdlib (flutter/packages#3960) 2023-05-12 [email protected] [local_auth] Improve iOS test DI (flutter/packages#3959) 2023-05-12 [email protected] [ci] Ensure scripts fail if a command fails (flutter/packages#3963) 2023-05-11 [email protected] [ci] Pin the Flutter version for `release` (flutter/packages#3965) 2023-05-11 [email protected] [video_player] foundation - reduce seek accuracy to fix seek to end bug (flutter/packages#3784) 2023-05-11 [email protected] [image_picker] Fix use_build_context_synchronously violations (flutter/packages#3969) 2023-05-11 [email protected] [tool] Ensure that publish credential path is available (flutter/packages#3970) 2023-05-11 [email protected] [go_router] change `fix_data.yaml` uris to be absolute to fix lints (flutter/packages#3877) 2023-05-11 [email protected] [tool] Use new pub cache location for publish (flutter/packages#3962) 2023-05-10 [email protected] Roll Flutter (stable) from f72efea to 84a1e90 (1920 revisions) (flutter/packages#3961) 2023-05-10 [email protected] [image_picker_android] Improved Bitmap resize on Android (flutter/packages#3423) 2023-05-10 [email protected] [camerax] Add Kotlin duplicate classes known error to README.md (flutter/packages#3957) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages-flutter-autoroll Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

…by target api 34 (#3973)" This reverts commit 5763d44.

…by target api 34" (#4027) Reverts #3973 flutter/flutter#127014 flutter/flutter#126460 (comment) https://stackoverflow.com/questions/76244218/flutter-url-launcher-plugin-throws-java-lang-illegalargumentexception-receiver

flutter/packages@6328cfb...6608b60 2023-05-15 [email protected] [ci] Remove unnecessary Linux libraries (flutter/packages#3975) 2023-05-15 [email protected] [ci] Enforce a minimum Kotlin version in examples (flutter/packages#3979) 2023-05-15 [email protected] [ci] Disable Windows repo tool tests (flutter/packages#3980) 2023-05-12 [email protected] [ci] Add LUCI repo tool tests (flutter/packages#3964) 2023-05-12 [email protected] [url_launcher] Set broadcast reciever visability as required by target api 34 (flutter/packages#3973) 2023-05-12 [email protected] Update build.gradle with a comment explaining version (flutter/packages#3972) 2023-05-12 [email protected] Roll Flutter from 8c5a1ea to a76dbe4 (12 revisions) (flutter/packages#3958) 2023-05-12 [email protected] [image_picker] Fix Android lints (flutter/packages#3887) 2023-05-12 [email protected] [various] Update minimum Flutter version to 3.3 (flutter/packages#3967) 2023-05-12 [email protected] [tool] Target specific Android unit tests (flutter/packages#3955) 2023-05-12 [email protected] [camerax] Add a dependency on kotlin-bom to align versions of kotlin-stdlib (flutter/packages#3960) 2023-05-12 [email protected] [local_auth] Improve iOS test DI (flutter/packages#3959) 2023-05-12 [email protected] [ci] Ensure scripts fail if a command fails (flutter/packages#3963) 2023-05-11 [email protected] [ci] Pin the Flutter version for `release` (flutter/packages#3965) 2023-05-11 [email protected] [video_player] foundation - reduce seek accuracy to fix seek to end bug (flutter/packages#3784) 2023-05-11 [email protected] [image_picker] Fix use_build_context_synchronously violations (flutter/packages#3969) 2023-05-11 [email protected] [tool] Ensure that publish credential path is available (flutter/packages#3970) 2023-05-11 [email protected] [go_router] change `fix_data.yaml` uris to be absolute to fix lints (flutter/packages#3877) 2023-05-11 [email protected] [tool] Use new pub cache location for publish (flutter/packages#3962) 2023-05-10 [email protected] Roll Flutter (stable) from f72efea to 84a1e90 (1920 revisions) (flutter/packages#3961) 2023-05-10 [email protected] [image_picker_android] Improved Bitmap resize on Android (flutter/packages#3423) 2023-05-10 [email protected] [camerax] Add Kotlin duplicate classes known error to README.md (flutter/packages#3957) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages-flutter-autoroll Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

…t api 34 (flutter#3973) - Update the version of androidx used. - Explicitly set Exported on previously exported broadcast reciever [126460](flutter/flutter#126460) This change is a no-op in behavior. I am open to the argument that instead we make this configurable. https://developer.android.com/guide/components/broadcasts#context-registered-receivers

flutter#3973 caused an out-of-band failure after publishing, because an example that uses `url_launcher` had a too-old Kotlin version set. This is not something we consider client-breaking because `flutter` prodives a very clear error message with a straightforward and actionable fix step (update the app's Kotlin version), so the fix for the breakage is just to update our own examples. Since increasingly we're likely to hit problems where modern version of dependencies don't work with old version of Kotlin, this adds repo-wide CI enforcement that examples are set to a minimum version (matching the current `flutter/flutter` template; we can increase this over time as we feel it's useful to do so).

…by target api 34" (flutter#4027) Reverts flutter#3973 flutter/flutter#127014 flutter/flutter#126460 (comment) https://stackoverflow.com/questions/76244218/flutter-url-launcher-plugin-throws-java-lang-illegalargumentexception-receiver

fixed [#flutter/flutter/issues/126460 ](flutter/flutter#126460) Reland of #3973 which incorrectly used application context when registering and not when unregistering. This pr uses activity context for both which is aligned with what the behavior was before. - Register on the same context as unregister - Add integration test for android url launch and close Tested with manual test and new integration test.

reidbaker added 2 commits May 12, 2023 12:30

Update the version of androidx used.

e45c050

Explictly set Exported on previously exported broadcast reciever

120ac9a

github-actions bot added p: url_launcher platform-android labels May 12, 2023

reidbaker requested review from stuartmorgan-g and camsim99 May 12, 2023 16:45

reidbaker marked this pull request as ready for review May 12, 2023 16:45

reidbaker requested a review from gmackall as a code owner May 12, 2023 16:45

flutter-dashboard bot added the needs tests label May 12, 2023

reidbaker added 2 commits May 12, 2023 12:55

Fix import order

253957d

independent version for core test

bea8585

stuartmorgan-g reviewed May 12, 2023

View reviewed changes

replace not add second registation

851fc86

stuartmorgan-g approved these changes May 12, 2023

View reviewed changes

reidbaker commented May 12, 2023

View reviewed changes

packages/url_launcher/url_launcher_android/CHANGELOG.md Outdated Show resolved Hide resolved

Update packages/url_launcher/url_launcher_android/CHANGELOG.md

cd26d39

reidbaker mentioned this pull request May 12, 2023

fix(battery_plus): Use context compat to set exported state fluttercommunity/plus_plugins#1811

Merged

7 tasks

reidbaker added the autosubmit Merge PR when tree becomes green via auto submit App label May 12, 2023

auto-submit bot merged commit 5763d44 into flutter:main May 12, 2023

reidbaker deleted the i126460-recievers-export-url-launcher branch May 12, 2023 21:01

stuartmorgan-g mentioned this pull request May 13, 2023

[ci] Enforce a minimum Kotlin version in examples #3979

Merged

engine-flutter-autoroll mentioned this pull request May 15, 2023

Roll Packages from 6328cfbd2a46 to 8febbba0c387 (21 revisions) flutter/flutter#126832

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 15, 2023

5763d44 [url_launcher] Set broadcast reciever visability as required …

29e82d6

…by target api 34 (flutter/packages#3973)

engine-flutter-autoroll mentioned this pull request May 15, 2023

Roll Packages from 6328cfbd2a46 to 8febbba0c387 (21 revisions) flutter/flutter#126859

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 15, 2023

5763d44 [url_launcher] Set broadcast reciever visability as required …

2cf008a

…by target api 34 (flutter/packages#3973)

engine-flutter-autoroll mentioned this pull request May 15, 2023

Roll Packages from 6328cfbd2a46 to 6608b604b0e7 (22 revisions) flutter/flutter#126865

Merged

engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 15, 2023

5763d44 [url_launcher] Set broadcast reciever visability as required …

947dd12

…by target api 34 (flutter/packages#3973)

huycozy mentioned this pull request May 17, 2023

url_launcher java.lang.RuntimeException: Unable to destroy activity, java.lang.IllegalArgumentException: Receiver not registered flutter/flutter#127014

Closed

2 tasks

reidbaker added a commit that referenced this pull request May 17, 2023

Revert "[url_launcher] Set broadcast reciever visability as required …

fc6ea1d

…by target api 34 (#3973)" This reverts commit 5763d44.

reidbaker mentioned this pull request May 17, 2023

Revert "[url_launcher] Set broadcast reciever visability as required by target api 34" #4027

Merged

This was referenced Jul 7, 2023

[Tool] New tool to download android dependencies #4408

Merged

[camera] tree fix format 2023 05 12 #4450

Closed

[tooling] mark infra steps as infra steps in yaml files #4473

Merged

reidbaker mentioned this pull request Jul 26, 2023

[Tooling] Add google owned cache for dependencies as an option in ci #4567

Merged

11 tasks

This was referenced Aug 4, 2023

[All] Expand artifact hub to all plugins #4645

Merged

[url_launcher] Android API 34 support #4660

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[url_launcher] Set broadcast reciever visability as required by target api 34 #3973

[url_launcher] Set broadcast reciever visability as required by target api 34 #3973

Uh oh!

reidbaker commented May 12, 2023 •

edited

Loading

Uh oh!

flutter-dashboard bot commented May 12, 2023

Uh oh!

stuartmorgan-g May 12, 2023

Uh oh!

reidbaker May 12, 2023

Uh oh!

stuartmorgan-g left a comment

Uh oh!

stuartmorgan-g May 12, 2023

Uh oh!

Uh oh!

Hixie commented May 12, 2023

Uh oh!

stuartmorgan-g commented May 13, 2023

Uh oh!

Uh oh!

		@@ -144,6 +145,11 @@ public void onCreate(@Nullable Bundle savedInstanceState) {

		// Register receiver that may finish this Activity.
		registerReceiver(broadcastReceiver, closeIntentFilter);

		@@ -1,3 +1,7 @@
		## 6.0.33

		* Explicitly set if reciever for close should be exported.

[url_launcher] Set broadcast reciever visability as required by target api 34 #3973

[url_launcher] Set broadcast reciever visability as required by target api 34 #3973

Uh oh!

Conversation

reidbaker commented May 12, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Pre-launch Checklist

Uh oh!

flutter-dashboard bot commented May 12, 2023

Uh oh!

stuartmorgan-g May 12, 2023

Choose a reason for hiding this comment

Uh oh!

reidbaker May 12, 2023

Choose a reason for hiding this comment

Uh oh!

stuartmorgan-g left a comment

Choose a reason for hiding this comment

Uh oh!

stuartmorgan-g May 12, 2023

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Hixie commented May 12, 2023

Uh oh!

stuartmorgan-g commented May 13, 2023

Uh oh!

Uh oh!

reidbaker commented May 12, 2023 •

edited

Loading