Skip to content

CMM-992 reader deeplinks #22374

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 10 commits into
base: release/26.4
Choose a base branch
from
+642 −40
Open

CMM-992 reader deeplinks #22374

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
205bc6e
Supporting read and discovery deeplinks
adalpari Nov 28, 2025
ba8e9dd
Fixing discovery issue
adalpari Nov 28, 2025
a13caf8
Handling feed links
adalpari Nov 28, 2025
f1aa2b4
Handling search and tags
adalpari Nov 28, 2025
4b98e77
Some improvements
adalpari Nov 28, 2025
53e19b9
Re-adding deleted code
adalpari Nov 28, 2025
5bbf6a7
Lint fixes
adalpari Nov 28, 2025
43c3cef
Adding tests for ReaderActivityLauncher
adalpari Nov 28, 2025
2625667
Update WordPress/src/main/java/org/wordpress/android/ui/ActivityLaunc…
adalpari Nov 28, 2025
9eb3e36
chore: trigger CI
adalpari Dec 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions WordPress/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -493,6 +493,8 @@ dependencies {
testImplementation(libs.assertj.core)
testImplementation(libs.kotlinx.coroutines.test)
testImplementation(libs.turbine)
testImplementation(libs.robolectric)
testImplementation(libs.androidx.test.core)

androidTestImplementation project(path:':libs:mocks')

Expand Down
114 changes: 106 additions & 8 deletions WordPress/src/main/AndroidManifest.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -581,6 +581,82 @@
android:pathPattern="/site-monitoring/.*"
android:scheme="http" />

<data
android:host="wordpress.com"
android:path="/read"
android:scheme="https" />

<data
android:host="wordpress.com"
android:path="/read"
android:scheme="http" />

<data
android:host="wordpress.com"
android:path="/discover"
android:scheme="https" />

<data
android:host="wordpress.com"
android:path="/discover"
android:scheme="http" />

<data
android:host="wordpress.com"
android:pathPattern="/read/feeds/.*"
Copy link

@claude claude bot Nov 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Security: Overly broad pathPattern: The pattern /read/feeds/.* will match ANY path starting with /read/feeds/, including potentially malicious paths like /read/feeds/../../../etc/passwd. While Android's intent filtering doesn't typically allow path traversal attacks, consider using more specific patterns.

The security scanner flagged these data tags as having non-unique attributes. While this is a warning, it might be cleaner to be more explicit with your intent filters. However, this is likely acceptable for your use case.

android:scheme="https" />

<data
android:host="wordpress.com"
android:pathPattern="/read/feeds/.*"
android:scheme="http" />

<data
android:host="wordpress.com"
android:pathPattern="/reader/feeds/.*"
android:scheme="https" />

<data
android:host="wordpress.com"
android:pathPattern="/reader/feeds/.*"
android:scheme="http" />

<data
android:host="wordpress.com"
android:path="/read/search"
android:scheme="https"
tools:ignore="IntentFilterUniqueDataAttributes" />

<data
android:host="wordpress.com"
android:path="/read/search"
android:scheme="http"
tools:ignore="IntentFilterUniqueDataAttributes" />

<data
android:host="wordpress.com"
android:path="/reader/search"
android:scheme="https"
tools:ignore="IntentFilterUniqueDataAttributes" />

<data
android:host="wordpress.com"
android:path="/reader/search"
android:scheme="http"
tools:ignore="IntentFilterUniqueDataAttributes" />

<data
android:host="wordpress.com"
android:pathPattern="/tag/.*"
android:scheme="https"
tools:ignore="IntentFilterUniqueDataAttributes" />

<data
android:host="wordpress.com"
android:pathPattern="/tag/.*"
android:scheme="http"
tools:ignore="IntentFilterUniqueDataAttributes" />

</intent-filter>
</activity-alias>

Expand Down Expand Up @@ -612,49 +688,71 @@
<data
android:host="wordpress.com"
android:pathPattern="/read/feeds/.*/posts/.*"
android:scheme="https" >
android:scheme="https"
tools:ignore="IntentFilterUniqueDataAttributes" >
Copy link
Contributor Author

@adalpari adalpari Nov 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moving the ignore suppression here instead of having a huuuge lint/baseline file

</data>

<data
android:host="wordpress.com"
android:pathPattern="/read/feeds/.*/posts/.*"
android:scheme="http" >
android:scheme="http"
tools:ignore="IntentFilterUniqueDataAttributes" >
</data>

<data
android:host="wordpress.com"
android:pathPattern="/read/blogs/.*/posts/.*"
android:scheme="https" >
android:scheme="https"
tools:ignore="IntentFilterUniqueDataAttributes" >
</data>

<data
android:host="wordpress.com"
android:pathPattern="/read/blogs/.*/posts/.*"
android:scheme="http" >
android:scheme="http"
tools:ignore="IntentFilterUniqueDataAttributes" >
</data>

<data
android:host="*.wordpress.com"
android:pathPattern="/2.../../../.*"
android:scheme="https" >
android:scheme="https"
tools:ignore="IntentFilterUniqueDataAttributes" >
</data>

<data
android:host="*.wordpress.com"
android:pathPattern="/2.../../../.*"
android:scheme="http" >
android:scheme="http"
tools:ignore="IntentFilterUniqueDataAttributes" >
</data>

<data
android:host="*.wordpress.com"
android:pathPattern="/19../../../.*"
android:scheme="https" >
android:scheme="https"
tools:ignore="IntentFilterUniqueDataAttributes" >
</data>

<data
android:host="*.wordpress.com"
android:pathPattern="/19../../../.*"
android:scheme="http" >
android:scheme="http"
tools:ignore="IntentFilterUniqueDataAttributes" >
</data>

<data
android:host="wordpress.com"
android:pathPattern="/reader/feeds/.*/posts/.*"
android:scheme="https"
tools:ignore="IntentFilterUniqueDataAttributes" >
</data>

<data
android:host="wordpress.com"
android:pathPattern="/reader/feeds/.*/posts/.*"
android:scheme="http"
tools:ignore="IntentFilterUniqueDataAttributes" >
</data>

<action android:name="org.wordpress.android.action.VIEW_POST" />
Expand Down
37 changes: 37 additions & 0 deletions WordPress/src/main/java/org/wordpress/android/ui/ActivityLauncher.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -338,6 +338,43 @@
context.startActivity(intent);
}

public static void viewReaderDiscoverInNewStack(Context context) {

Check notice

Code scanning / Android Lint

Nullable/NonNull annotation missing on method parameter Note

Missing null annotation
Intent intent = getMainActivityInNewStack(context);
intent.putExtra(WPMainActivity.ARG_OPEN_PAGE, WPMainActivity.ARG_READER);
intent.putExtra(WPMainActivity.ARG_READER_DISCOVER_TAB, true);
context.startActivity(intent);
}

public static void viewReaderFeedInNewStack(Context context, long feedId) {
Intent mainActivityIntent = getMainActivityInNewStack(context)
.putExtra(WPMainActivity.ARG_OPEN_PAGE, WPMainActivity.ARG_READER);
Intent feedIntent = ReaderActivityLauncher.buildReaderFeedIntent(context, feedId, "deeplink");
TaskStackBuilder.create(context)
.addNextIntent(mainActivityIntent)
.addNextIntent(feedIntent)
.startActivities();
}

public static void viewReaderSearchInNewStack(Context context) {
Intent mainActivityIntent = getMainActivityInNewStack(context)
.putExtra(WPMainActivity.ARG_OPEN_PAGE, WPMainActivity.ARG_READER);
Intent searchIntent = ReaderActivityLauncher.createReaderSearchIntent(context);
TaskStackBuilder.create(context)
.addNextIntent(mainActivityIntent)
.addNextIntent(searchIntent)
.startActivities();
}

public static void viewReaderTagInNewStack(@NonNull Context context, @NonNull String tagSlug) {
Intent mainActivityIntent = getMainActivityInNewStack(context)
.putExtra(WPMainActivity.ARG_OPEN_PAGE, WPMainActivity.ARG_READER);
Intent tagIntent = ReaderActivityLauncher.buildReaderTagIntent(context, tagSlug, "deeplink");
TaskStackBuilder.create(context)
.addNextIntent(mainActivityIntent)
.addNextIntent(tagIntent)
.startActivities();
}

public static void viewPostDeeplinkInNewStack(Context context, Uri uri) {
Intent mainActivityIntent = getMainActivityInNewStack(context)
.putExtra(WPMainActivity.ARG_OPEN_PAGE, WPMainActivity.ARG_READER);
Expand Down Expand Up @@ -515,7 +552,7 @@

public static PendingIntent buildStatsPendingIntentOverMainActivityInNewStack(Context context, SiteModel site,
@Nullable StatsTimeframe timeframe,
@Nullable String period,

Check notice

Code scanning / Android Lint

Nullable/NonNull annotation missing on method parameter Note

Missing null annotation
@Nullable NotificationType type,
@NonNull
StatsLaunchedFrom launchedFrom,
Expand Down Expand Up @@ -1546,7 +1583,7 @@
intent.putExtra(SignupEpilogueActivity.EXTRA_SIGNUP_USERNAME, username);
intent.putExtra(SignupEpilogueActivity.EXTRA_SIGNUP_IS_EMAIL, isEmail);
activity.startActivityForResult(intent, RequestCodes.SHOW_SIGNUP_EPILOGUE_AND_RETURN);
}

Check notice

Code scanning / Android Lint

Nullable/NonNull annotation missing on method parameter Note

Missing null annotation

public static void showPostSignupInterstitial(Context context) {
final Intent parentIntent = new Intent(context, WPMainActivity.class);
Expand Down
12 changes: 12 additions & 0 deletions WordPress/src/main/java/org/wordpress/android/ui/deeplinks/DeepLinkNavigator.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,11 @@
import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenPages
import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenPagesForSite
import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenQRCodeAuthFlow
import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenFeedInReader
import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenReader
import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenReaderDiscover
import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenReaderSearch
import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenTagInReader
import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenStats
import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenStatsForSite
import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenStatsForSiteAndTimeframe
Expand All @@ -37,7 +41,7 @@
@Inject constructor(private val activityNavigator: ActivityNavigator) {
@Suppress("ComplexMethod", "LongMethod")
fun handleNavigationAction(navigateAction: NavigateAction, activity: AppCompatActivity) {
when (navigateAction) {

Check warning on line 44 in WordPress/src/main/java/org/wordpress/android/ui/deeplinks/DeepLinkNavigator.kt

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Reduce the number of when branches from 32 to at most 30. 


See more on https://sonarcloud.io/project/issues?id=wordpress-mobile_WordPress-Android&issues=AZrZoGbWx9bj30CvRc7f&open=AZrZoGbWx9bj30CvRc7f&pullRequest=22374
LoginForResult -> ActivityLauncher.loginForDeeplink(activity)
StartCreateSiteFlow -> {
ActivityLauncher.showMainActivityAndSiteCreationActivity(activity, DEEP_LINK)
Expand Down Expand Up @@ -78,6 +82,10 @@
)

OpenReader -> ActivityLauncher.viewReaderInNewStack(activity)
OpenReaderDiscover -> ActivityLauncher.viewReaderDiscoverInNewStack(activity)
OpenReaderSearch -> ActivityLauncher.viewReaderSearchInNewStack(activity)
is OpenFeedInReader -> ActivityLauncher.viewReaderFeedInNewStack(activity, navigateAction.feedId)
is OpenTagInReader -> ActivityLauncher.viewReaderTagInNewStack(activity, navigateAction.tagSlug)
is OpenInReader -> ActivityLauncher.viewPostDeeplinkInNewStack(activity, navigateAction.uri.uri)
is ViewPostInReader -> ActivityLauncher.viewReaderPostDetailInNewStack(
activity,
Expand Down Expand Up @@ -123,6 +131,10 @@
data class OpenEditorForPost(val site: SiteModel, val postId: Int) : NavigateAction()
data class OpenEditorForSite(val site: SiteModel) : NavigateAction()
object OpenReader : NavigateAction()
object OpenReaderDiscover : NavigateAction()
object OpenReaderSearch : NavigateAction()
data class OpenFeedInReader(val feedId: Long) : NavigateAction()
data class OpenTagInReader(val tagSlug: String) : NavigateAction()
data class OpenInReader(val uri: UriWrapper) : NavigateAction()
data class ViewPostInReader(val blogId: Long, val postId: Long, val uri: UriWrapper) : NavigateAction()
object OpenEditor : NavigateAction()
Expand Down
Loading