[Geolocalisation] Impossible de partager sa geoloc en live pour un invité

[NicolasBuquet]

Un utilisateur ayant invité dans une DM un autre utilisateur, ce dernier ne peut pas partager sa géolocalisation pour cause de droit insuffisant dans la room.

En effet, le droit de l'utilisateur invité est au niveau 0, alors qu'il faut au moins être Modérateur (niveau 50) pour pouvoir partager sa position.

le créateur est niveau 100 (admin).

Pourquoi un utilisateur rejoingnat une DM n'est pas administrateur de cette DM lui aussi (niveau 100) ?

[NicolasBuquet]

Le code responsable de l'erreur :

    // Check if user can send beacon info state event
    private func canShareLiveLocation() -> Bool {
        guard let myUserId = parameters.roomDataSource.mxSession.myUserId else {
            return false
        }
        
        let userPowerLevelRawValue = parameters.roomDataSource.roomState.powerLevels.powerLevelOfUser(withUserID: myUserId)
        
        guard let userPowerLevel = RoomPowerLevel(rawValue: userPowerLevelRawValue) else {
            return false
        }
        
        return userPowerLevel.rawValue >= RoomPowerLevel.moderator.rawValue
    }

[NicolasBuquet]

@mcalinghee @odelcroi

Même bug de userLevel que ce ticket : https://github.com/tchapgouv/tchap-backend/issues/18 ?

[odelcroi]

C'est uniquement pour les partage de position en direct n'est ce pas?
Si oui, alors je le preciserai dans le titre du ticket, de la PR et de la release note

[NicolasBuquet]

@odelcroi
Oui, uniquement en live sharing.

[NicolasBuquet]

Suite à un point ce jour avec @MatMaul et @MarcWadai, voici la situation.

• Ça concerne bien aussi les rooms classiques : un utilisateur invité n'a pas de droits assez élevés pour partager sa géolocalisation en temps réel.
• C'est patché en DEV :
  • les rooms nouvellement créées le sont avec les bons droits :
  • admin en DM pour tout le monde
  • le bon droit par défault sur le partage de géoloc sur les rooms classiques qui permet à un invité de partager en temps réel
• Script de mise à jour des DM et rooms classiques existantes a été testé en DEV. Il est OK.

[odelcroi]

tracké ici : https://github.com/tchapgouv/tchap-backend/issues/38

[NicolasBuquet]

Test geoloc effectué ce jour (16/05/2024) avec @yostyle et @MarcWadai sur l'environnement DEV :

• DM :

  • Android-iOS : ok
  • iOS-Android : ok
  • web-iOS : ok
  • iOS-web : ok
  • web-android : ok
  • android-web : ok

• Room nouvelle incluant le patch de @MatMaul à la création :

  • web : ok
  • android: ok
  • iOS : ko (nécessite d'être Modérateur au moins - restriction code côté client iOS)

• Room d'avant patch Mathieu :

  • web : ok
  • android: ok
  • iOS : ko (nécessite d'être Modérateur au moins - restriction code côté client iOS)

Le patch de @MatMaul fonctionne bien pour Android et web.

On va donc intégrer le patch iOS sur la non-vérification de droit minimum pour partager en temps réel sa géolocalisation.