Par défaut les moteurs web restreignent la possibilité de faire des requêtes d'un site à un autre (Cross-Origin Resource Sharing) seulement si le site en question le permet, y compris pour des pages hébergées localement, donc la plupart des requêtes effectuées par l'app aux services scolaires passent actuellement par un serveur intermédiaire nommé proxy CORS.

Cette PR implémente le patch des fonctions Fetch et XMLHttpRequest (utilisé par Axios) fourni par le plugin Http de Capacitor, afin d'utiliser, sur les plateformes natives (iOS et Android), le système de requêtes HTTP natif qui n'est pas soumis aux restrictions CORS.

Cela a un impact sur l'ensemble de l'application, et pourrait potentiellement provoquer des bugs inédits.

Fonctionnalités concernées à tester

• Toute fonctionnalité impliquant une requête réseau
• en particulier celles normalement soumises aux restrictions CORS :
  • Toute requête vers Skolengo/Kdecole
  • Pronote : chargement de photo de profil
  • Pronote : recherche d'établissement par code postal et localisation
  • Envoi de logs

Progression

• Test rapide sur web (rien ne change vraiment en interne, juste pour vérifier que je n'ai pas fait d'erreur)
• Build et test sur Android
• Écrire un changelog stylé
• Demander plus de testeurs iOS et Android

Pull request checklist

Please check if your PR fulfills the following requirements:

• You can build successfully the whole project with your changes locally
• You use the coding conventions and the naming conventions of the project
• You use tabs for indentation
• You make sure that your PR is not a duplicate
• This PR is ready to be reviewed and merged
• This PR merges into the development branch or in the specific branch of the feature you want to merge into
• There is no TODO in the code
• There is no spelling or grammatical errors in the code
• Details of the issue or feature are documented below
• This PR is not a breaking change (e.g. changes that would cause existing functionality to change)

Proposed changelog

Note

Before merging this PR, you will need the approval of at least one of the following people:

• @ecnivtwelve
• @lucas-luchack

So be patient ;)

Informations supplémentaires

close #112