Skip to content

[auth] Add search-by-intent via ente-auth://search?query= deep link #9501

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
bekoeppel wants to merge 3 commits into
base: main
Choose a base branch
from
+31 −17
Open

[auth] Add search-by-intent via ente-auth://search?query= deep link #9501

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
40332ed
[auth] Add search-by-intent via ente-auth://search?query= deep link
bekoeppel Mar 4, 2026
17ee46b
[auth] Guard ente-auth deep link parsing against malformed URIs
bekoeppel Mar 4, 2026
9fceb8f
[auth] Skip throttle for search deep links
bekoeppel Mar 4, 2026
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
34 changes: 22 additions & 12 deletions mobile/apps/auth/lib/ui/home_page.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,7 @@ class _HomePageState extends State<HomePage> {
List<String> tags = [];
List<Code> _filteredCodes = [];
StreamSubscription<CodesUpdatedEvent>? _streamSubscription;
StreamSubscription<String>? _deepLinkSubscription;
StreamSubscription<TriggerLogoutEvent>? _triggerLogoutEvent;
StreamSubscription<IconsChangedEvent>? _iconsChangedEvent;
StreamSubscription<MultiSelectActionRequestedEvent>?
Expand Down Expand Up @@ -1191,6 +1192,7 @@ class _HomePageState extends State<HomePage> {
@override
void dispose() {
_streamSubscription?.cancel();
_deepLinkSubscription?.cancel();
_triggerLogoutEvent?.cancel();
_iconsChangedEvent?.cancel();
_multiSelectActionSubscription?.cancel();
Expand Down Expand Up @@ -1865,29 +1867,26 @@ class _HomePageState extends State<HomePage> {
}
}

late final AppLinks _appLinks = AppLinks();

Future<bool> _initDeepLinks() async {
// Platform messages may fail, so we use a try/catch PlatformException.
final appLinks = AppLinks();
bool hadInitialLink = false;
try {
String? initialLink;
initialLink = await appLinks.getInitialLinkString();
// Parse the link and warn the user, if it is not correct,
// but keep in mind it could be `null`.
final initialLink = await _appLinks.getInitialLinkString();
if (initialLink != null) {
_handleDeeplink(context, initialLink);
return true;
hadInitialLink = true;
} else {
_logger.info("No initial link received.");
}
} on PlatformException {
// Handle exception by warning the user their action did not succeed
// return?
_logger.severe("PlatformException thrown while getting initial link");
}

// Attach a listener to the stream
// Always attach a listener for future deep links
if (!kIsWeb && !Platform.isLinux) {
appLinks.stringLinkStream.listen(
_deepLinkSubscription = _appLinks.stringLinkStream.listen(
(link) {
_handleDeeplink(context, link);
},
Expand All @@ -1896,7 +1895,7 @@ class _HomePageState extends State<HomePage> {
},
);
}
return false;
return hadInitialLink;
}

int lastScanTime = DateTime.now().millisecondsSinceEpoch - 1000;
Expand All @@ -1913,7 +1912,18 @@ class _HomePageState extends State<HomePage> {
return;
}
lastScanTime = DateTime.now().millisecondsSinceEpoch;
if (mounted && link.toLowerCase().startsWith("otpauth://")) {
final lowerLink = link.toLowerCase();
if (mounted && (lowerLink.startsWith("ente-auth://") || lowerLink.startsWith("enteauth://"))) {
Copy link
Copy Markdown
Contributor

@prateekmedia prateekmedia Mar 22, 2026

Choose a reason for hiding this comment

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

I am not sure about ente-auth:// pattern, any reason you included it here?

Copy link
Copy Markdown
Author

@bekoeppel bekoeppel Mar 29, 2026

Choose a reason for hiding this comment

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

I saw ente-auth:// pattern (with a -) is registered in AndroidManifest.xml:

ente/mobile/apps/auth/android/app/src/main/AndroidManifest.xml

Lines 19 to 24 in 164efcf

<intent-filter>
<action android:name="android.intent.action.VIEW"/>
<category android:name="android.intent.category.DEFAULT"/>
<category android:name="android.intent.category.BROWSABLE"/>
<data android:scheme="ente-auth"/>
</intent-filter>

along with the enteauth:// pattern (without a -):

ente/mobile/apps/auth/android/app/src/main/AndroidManifest.xml

Lines 33 to 38 in 164efcf

<intent-filter>
<action android:name="android.intent.action.VIEW" />
<category android:name="android.intent.category.DEFAULT" />
<category android:name="android.intent.category.BROWSABLE" />
<data android:scheme="enteauth" />
</intent-filter>

Let me know if I should remove the one with the -.

final uri = Uri.parse(link);
if (uri.host != "search") return;
final searchQuery = uri.queryParameters['query'];
if (searchQuery != null && searchQuery.isNotEmpty) {
_showSearchBox = true;
_textController.text = searchQuery;
_searchText = searchQuery;
_applyFilteringAndRefresh();
}
} else if (mounted && lowerLink.startsWith("otpauth://")) {
try {
final newCode = Code.fromOTPAuthUrl(link);
getNextTotp(newCode);
Expand Down