Skip to content

Commit f34d1e7

Browse files
authored
Fix: API 30+ fix for opening Pinpoint push notification links (#2924)
* Fix for opening links above api 30 * add exception to log
1 parent f34c5cb commit f34d1e7

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

aws-android-sdk-pinpoint/src/main/java/com/amazonaws/mobileconnectors/pinpoint/targeting/notification/NotificationClientBase.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
import android.app.Notification;
1919
import android.app.NotificationManager;
2020
import android.app.PendingIntent;
21+
import android.content.ActivityNotFoundException;
2122
import android.content.Context;
2223
import android.content.Intent;
2324
import android.content.pm.ApplicationInfo;
@@ -812,8 +813,12 @@ private void openURL(final String url, final boolean noSchemeValidation) {
812813
final Intent intent = new Intent(Intent.ACTION_VIEW);
813814
intent.setData(Uri.parse(validatedUrl));
814815
intent.setFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
815-
if (intent.resolveActivity(pinpointContext.getApplicationContext().getPackageManager()) != null) {
816+
817+
// Querying packages now requires query manifest flag, so we instead try/catch the attempt
818+
try {
816819
pinpointContext.getApplicationContext().startActivity(intent);
820+
} catch (ActivityNotFoundException e) {
821+
log.error("Couldn't find an app to open ACTION_VIEW Intent.", e);
817822
}
818823
}
819824

0 commit comments

Comments
 (0)