ability to rewrite location and cookies on v1-toolkit proxies
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 25/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- typescript
- Domaine
- cli
Piste de recherche
Commencez par la configuration du proxy dans ionic.config.json et examinez le fork v1-toolkit-proxy-rewrite-location-and-cookie associé. Vérifiez comment la CLI utilise http-proxy-middleware et déterminez si la réécriture du domaine des cookies, la réécriture de Location, le débogage et les cibles de proxy dynamiques basées sur le chemin peuvent être regroupés dans le même périmètre. La tâche est terminée lorsqu’un périmètre fonctionnel convenu et une implémentation acceptée pour les comportements de proxy sélectionnés sont disponibles.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Our app runs ionic v1 and until now we used Chrome with some security settings turned off when testing our app in order to avoid most cross-origin issues like CORS and cookie samesite over HTTP when we call our REST API.
Recently Chrome 94 dropped support of some flags we really need like SameSiteByDefaultCookies or CookiesWithoutSameSiteMustBeSecure.
So we decided to use ionic's proxies feature which is configurable in ionic.config.json.
Unfortunately we went through some impediments:
- one of our api does some redirection (like
http://testserver302-redirects tohttp://testserver/otherapi) - on production servers, some cookies have a domain set, which of course won't match the local test server entry point (usually http://localhost:8100)
- we have a lot of different test servers, this would require a lot of entries in the proxies array.
To cover cases 1) and 2), we implemented new settings for the proxy (development has started on a fork):
cookieDomainRewrite: { [domain: string]: string };turns on rewriting cookie domain on set-cookie headerslocationRewrite: booleanturns on rewriting location host+port AND path on redirection. For example if you proxy/apitohttp://testserverthen a headerLocation: http://testserver/otherapiwould be rewritten toLocation: http://localhost:8100/api/otherapi.debug: booleanturns on verbose debugging ofhttp-proxy-middleware
I know v1 is quite old, but could these features be of interest for other users and so we might open a pull request ? Or maybe you have some suggestions ?
For case 3) we are willing to implement dynamic proxy based on path. The idea is to redirect on a different server based on a part of the path. Something that would be configurable this way:
{
"path": "/api/*",
"proxyUrl": "http://$1/"
}
- Langage dominant
- TypeScript
- Étoiles
- 2k
- Forks
- 681
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de ionic-team/ionic-cli
-
triage
Difficulté 1/5 Moins d'une heure Accessibilité débutants 65/100
ionic-team/ionic-cli#5016 ·
-
ionic-cli development status Ouvertetriage
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 15/100
ionic-team/ionic-cli#5131 ·
-
triage
Difficulté 3/5 1-2 jours Accessibilité débutants 56/100
ionic-team/ionic-cli#5130 ·
-
triage
Difficulté 3/5 1-2 jours Accessibilité débutants 72/100
ionic-team/ionic-cli#5129 ·
-
triage
Difficulté 3/5 1-2 jours Accessibilité débutants 42/100
ionic-team/ionic-cli#5127 · 2 commentaires ·
Toutes les issues de ionic-team/ionic-cli
Issues similaires
-
bug(cli): hapi doctor inline-media prints a fabricated B:\ helper-script path in packaged installs Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
-
Crush Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 85/100
catppuccin/catppuccin#3125 ·
-
Add a SECURITY.md Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
ElementsProject/cln-application#167 · 1 commentaire · 1 réaction ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
Quantco/pnpm-licenses#17 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100