Skip to content
This repository has been archived by the owner on Oct 15, 2023. It is now read-only.

[WIP - DO NOT MERGE] Contourner les CORS en utilisant l'API native #114

Draft
wants to merge 3 commits into
base: development
Choose a base branch
from

Conversation

moeenio
Copy link
Contributor

@moeenio moeenio commented May 5, 2023

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

Français fr
Proposez un changelog en français ici...

Note

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

So be patient ;)

Informations supplémentaires

close #112

@moeenio
Copy link
Contributor Author

moeenio commented May 27, 2023

J'avais pas réussi à configurer le live reload de l'appli mobile sur Android (https://capacitorjs.com/docs/guides/live-reload) et je me suis pas repenché dessus depuis. @LeGeek01 si tu veux aider sur ça utilise cette branche

@moeenio moeenio marked this pull request as draft May 27, 2023 17:11
@ecnivtwelve
Copy link
Contributor

ecnivtwelve commented Jun 1, 2023

J'avais pas réussi à configurer le live reload de l'appli mobile sur Android (https://capacitorjs.com/docs/guides/live-reload) et je me suis pas repenché dessus depuis. @LeGeek01 si tu veux aider sur ça utilise cette branche

Si tu veux live reload (peu importe l'OS) fais le manuellement avec la config au lieu de la CLI qui marche une fois sur 12

@moeenio
Copy link
Contributor Author

moeenio commented Jun 1, 2023

comment

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

Successfully merging this pull request may close these issues.

[Enhancement]: se débarasser du proxy CORS
2 participants