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

Update download function to download from the right source #500

Open
Jean-BaptisteC opened this issue Feb 16, 2022 · 6 comments
Open

Update download function to download from the right source #500

Jean-BaptisteC opened this issue Feb 16, 2022 · 6 comments
Labels
question Further information is requested

Comments

@Jean-BaptisteC
Copy link
Contributor

Jean-BaptisteC commented Feb 16, 2022

Issue detected on submit function on Exodus app (2.2.0)

Change detection function and download rules

If source = Google Play and app id match on Google Play

  • Then download from Google Play

If source = Google Play and app id match on Fdroid

  • Then download from Fdroid

If source = Fdroid and app id match on Fdroid

  • Then download from Fdroid

If source = Fdroid and app id match on Google Play

  • Then download from Google Play

If source = Google Play or Fdroid and app id no match

  • Then return The application cannot be found

Algorithm can be optimize

@Jean-BaptisteC
Copy link
Contributor Author

If source = Google Play
  If app id match on Google Play
    Then download from Google Play
  If app id match on Fdroid
    Then download from Fdroid
  Else (no match app id)
    Then return The application cannot be found
Else (source = Fdroid)
  If app id match on Fdroid
    Then download from Fdroid
  If app id match on Google Play
    Then download from Google Play
  Else (no match app id)
    Then return The application cannot be found

@pnu-s
Copy link
Member

pnu-s commented Feb 17, 2022

I'm not sure we want to do that, I think we need to respect the user's choice of source.

If the user says they want to check an app from Google Play, I'm not sure we want to try downloading this app from F-Droid (that would imply a lot of unnecessary downloads). Beside, which report do we send back if we start 2 analysis in the same time.

I think what we need to do is that the android app should tell us the source.

It could be by redirecting to https://reports.exodus-privacy.eu.org/analysis/submit/#<full_url>

For instance https://reports.exodus-privacy.eu.org/en/analysis/submit/#https://f-droid.org/packages/org.eu.exodus_privacy.exodusprivacy/

Currently that does not work perfectly in the exodus website but can be fixed quickly

@pnu-s pnu-s added the question Further information is requested label Feb 17, 2022
@Jean-BaptisteC
Copy link
Contributor Author

Ok you can change for solution more easy
If source = Google Play
If app id match on Google Play
Then download from Google Play
Else (no match app id)
Then return The application cannot be found on GP
Else (source = Fdroid)
If app id match on Fdroid
Then download from Fdroid
Else (no match app id)
Then return The application cannot be found on fdroid

@pnu-s
Copy link
Member

pnu-s commented Feb 17, 2022

To make sure I understand your proposal, what you mean is that we could check that an app exists in the desired store (F-Droid or Google Play) before trying to download it?

@Jean-BaptisteC
Copy link
Contributor Author

Yes, actually you check with function validate_handle if app matched on Google Play and Fdroid. But like explain in past. If handle doesn't match, on Fdroid, exodus doesn't check if handle match on Google Play.

@yoshimo
Copy link

yoshimo commented Sep 28, 2022

Don't forget that some applications are on more than one store and maybe slightly different so whenever you submit an app it has to be downloaded and scanned for all stores.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants