OpenAPITools / OpenAPITools/openapi-generator
[REQ] Allow to pass OkHttpClient into ApiClient constructor
Personne n'a encore pris cette issue.
- Langage dominant
- Java
- Étoiles
- 26.8k
- Forks
- 7.7k
- Métriques de merge des PR
- Métriques de PR en attente
Description
The problem
Right now ApiClient for language java and library okhttp-gson is generated so that it always create a new OkHttpClient, which is not efficient. And it also recommended to have shared instance of OkHttpClient
In my app I'm go to call API, which is the same for different hosts, so I'm enforced to create ApiClient per host. And it creates OkHttpClient per host, which is I want to avoid.
Also it would be great to have possibility to provide own implementation for authentication.
Suggestion
Make it possible to pass OkHttpClient into constructor of ApiClient. Or maybe just not to instantiate it in ApiClient at all and set it via setter. But it's not very clean.
Make it possible to specify own implementation for authorization.
Also
I saw similar issue for WebClient.
https://github.com/OpenAPITools/openapi-generator/issues/874
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.
Piste de recherche
Commencez par le constructeur généré de Java okhttp-gson ApiClient et examinez comment il crée le OkHttpClient et gère l’authentification. Le travail est terminé lorsque les clients générés peuvent réutiliser un client fourni et prendre en charge une autorisation personnalisée tout en préservant le comportement par défaut existant.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- java
- Domaine
- api, tooling
- Type d'issue
- Fonctionnalité
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100