swagger-api/swagger-codegen

[swift] keep-alive not working

Open

#4.333 geöffnet am 7. Dez. 2016

Auf GitHub ansehen
 (9 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)HTML (5.474 Forks)batch import
Client: SwiftIssue: Bughelp wanted

Repository-Metriken

Stars
 (12.701 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Description

It looks like the way the Alamofire is implemented we loosing all of the keep-alive sessions because every request has it's own manager. Is there is a way to have keep-alive enabled that would greatly reduce latency on SSL connections. The code in question is this one https://github.com/swagger-api/swagger-codegen/blob/master/samples/client/petstore/swift/default/PetstoreClient/Classes/Swaggers/AlamofireImplementations.swift#L53

Contributor Guide

Research-Richtung
Das Problem verweist auf Zeile 53 in AlamofireImplementations.swift, wo für jede Anfrage ein neuer Manager erstellt wird, wodurch Keep Alive verloren geht. Untersuchen Sie, wie ein Alamofire SessionManager über Anfragen hinweg geteilt werden kann, um Wiederverwendung von Verbindungen zu ermöglichen. Prüfen Sie, ob es in den Kommentaren Diskussionen über vorgeschlagene Lösungen oder PRs gibt. Überprüfen Sie außerdem, ob der generierte Code seit der Meldung des Problems aktualisiert wurde.
Tech Stack
swift
Domain
api
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
SwiftAlamofire
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75