Skip to content
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.

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

feat: Serve well-known assetlinks json #44665

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Pytal
Copy link
Member

@Pytal Pytal commented Apr 4, 2024

Summary

Checklist

@Pytal Pytal added this to the Nextcloud 30 milestone Apr 4, 2024
@Pytal Pytal requested a review from alperozturk96 April 4, 2024 23:49
@Pytal Pytal self-assigned this Apr 4, 2024
@Pytal Pytal requested a review from artonge April 4, 2024 23:50
@Pytal
Copy link
Member Author

Pytal commented Apr 4, 2024

cc @alperozturk96 feel free to edit the assetlinks.json in the second commit

@alperozturk96
Copy link

cc @alperozturk96 feel free to edit the assetlinks.json in the second commit

We need to put "sha256_cert_fingerprints" please don't merge, until we put it. @tobiasKaminsky Can you edit assetlinks.json?

@kesselb
Copy link
Contributor

kesselb commented Apr 5, 2024

package_name and sha256_cert_fingerprints should be configurable via config.php similar to https://github.com/nextcloud/server/blob/master/config/config.sample.php#L1105-L1123 for branded clients.

@Pytal
Copy link
Member Author

Pytal commented Apr 6, 2024

package_name and sha256_cert_fingerprints should be configurable via config.php similar to master/config/config.sample.php#L1105-L1123 for branded clients.

Makes sense 👍

Would we need to allow multiple package_name and sha256_cert_fingerprints @tobiasKaminsky?

Signed-off-by: Christopher Ng <chrng8@gmail.com>
@tobiasKaminsky
Copy link
Member

Would we need to allow multiple package_name and sha256_cert_fingerprints @tobiasKaminsky?

@alperozturk96 can you check if this is possible or needed?

@alperozturk96
Copy link

alperozturk96 commented Apr 15, 2024

@Pytal assetlinks-json can contain multiple package names. @tobiasKaminsky as far as I can see we only have "com.nextcloud.client" this package name. Shouldn't be needed for another one.

Example

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Associating website with Android app For Better DeepLink Support
4 participants