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

antivirus : sur ios l'antivirus n'est pas appelé sur un fichier infecté #887

Closed
NicolasBuquet opened this issue Oct 9, 2023 · 1 comment · Fixed by #994
Closed

antivirus : sur ios l'antivirus n'est pas appelé sur un fichier infecté #887

NicolasBuquet opened this issue Oct 9, 2023 · 1 comment · Fixed by #994
Labels
bug Something isn't working iOS

Comments

@NicolasBuquet
Copy link
Contributor

Un fichier infecté est posté sur un salon.

Ce fichier est vu comme infecté sur web :
Screenshot 2023-10-09 at 18 37 06

Ce même fichier n'est pas vu comme infecté sur iOS :
Screenshot 2023-10-09 at 18 37 34

Mais un autre utilisateur iOS rapporte que ce fichier est vu infecté sur son iPhone :
Screenshot 2023-10-09 at 18 39 15

Sur iOS, la demande de scan antivirus est fait au ScanManager qui est une propriété de la session MXSession.
L'instanciation du scanManager est conditionnée au renseignement de la variable _antivirusServerURL dans le MatrixSDK :

- (void)setAntivirusServerURL:(NSString *)antivirusServerURL
{
    _antivirusServerURL = antivirusServerURL;
    
    // Update the current restClient
    [matrixRestClient setAntivirusServer:antivirusServerURL];
    
    // Configure scan manager if antivirusServerURL is set
    if (antivirusServerURL)
    {
        _scanManager = [[MXScanManager alloc] initWithRestClient:matrixRestClient];
        [_scanManager resetAllAntivirusScanStatusInProgressToUnknown];
    }
    else
    {
        _scanManager = nil;
    }
    
    // Update the media manager
    [mediaManager setScanManager:_scanManager];
}

L'application essaie de trouver cette valeur antivirusServerURL dans les réglages de l'application lors de l'appel MXAccountData:initWithCoder :

        if ([coder decodeObjectForKey:@"antivirusserverurl"])
        {
            _antivirusServerURL = [coder decodeObjectForKey:@"antivirusserverurl"];
        }

Questions ?

  • Qui initialise cette valeur et quand ?
  • Comment se fait-il que différentes configurations puissent avoir lieu pour différentes personnes ?
@NicolasBuquet NicolasBuquet added bug Something isn't working iOS labels Oct 9, 2023
@NicolasBuquet NicolasBuquet moved this to En étude in Roadmap Produit Oct 9, 2023
@odelcroi odelcroi changed the title L'antivirus n'est pas appelé sur un fichier infecté antivirus : sur ios l'antivirus n'est pas appelé sur un fichier infecté Oct 10, 2023
@odelcroi odelcroi moved this from En étude to sprint en cours - WIP in Roadmap Produit Nov 28, 2023
@NicolasBuquet NicolasBuquet moved this from sprint en cours - WIP to prêt pour le sprint in Roadmap Produit Dec 5, 2023
@NicolasBuquet
Copy link
Contributor Author

NicolasBuquet commented Mar 21, 2024

Le well-known n'a pas l'air de renseigner l'URL de l'antivirus.

Regarder du côté Android comment est renseignée cette URL.

Sur Android, antivirusServerUrl est initialisé à la même valeur que homeServerUrl quand le homeServer est initialisé.

https://github.com/tchapgouv/tchap-android/blob/c433cfe03720001c3dfca582eb02e5aff2cb22e1/matrix-sdk-android/src/main/java/org/matrix/android/sdk/api/auth/data/HomeServerConnectionConfig.kt#L42

@github-project-automation github-project-automation bot moved this from prêt pour le sprint to déployé en prod in Roadmap Produit Mar 25, 2024
@NicolasBuquet NicolasBuquet moved this from déployé en prod to sprint en cours - WIP in Roadmap Produit Mar 25, 2024
@NicolasBuquet NicolasBuquet moved this from sprint en cours - WIP to beta externe in Roadmap Produit Apr 3, 2024
@NicolasBuquet NicolasBuquet moved this from beta externe to déployé en prod in Roadmap Produit Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working iOS
Projects
Status: déployé en prod
Development

Successfully merging a pull request may close this issue.

1 participant