OpenAPITools / OpenAPITools/openapi-generator
[REQ] Allow to pass OkHttpClient into ApiClient constructor
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Java
- Sterne
- 26.8k
- Forks
- 7.7k
- PR-Merge-Kennzahlen
- PR-Kennzahlen ausstehend
Beschreibung
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
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne beim generierten Konstruktor von Java okhttp-gson ApiClient und untersuche, wie er den OkHttpClient erstellt und die Authentifizierung handhabt. Die Aufgabe ist erledigt, wenn generierte Clients einen bereitgestellten Client wiederverwenden und eine benutzerdefinierte Autorisierung unterstützen können, wobei das bestehende Standardverhalten erhalten bleibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java
- Bereich
- api, tooling
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100