swagger-api/swagger-codegen

[Swift] Add configuration class

Open

#5,642 opened on May 15, 2017

View on GitHub
 (3 comments) (0 reactions) (0 assignees)HTML (5,474 forks)batch import
Enhancement: FeatureGeneral: Discussionhelp wanted

Repository metrics

Stars
 (12,701 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

Description

Similar to ObjC API client, which uses DefaultConfiguration class to store settings such as username, password, host, option to verify SSL, etc, we want to consider doing the same in the Swift 2.x/3.x API client.

Currently, ObjC is using singleton for the configuration class. For Python, we just switched from singleton to per-ApiClient for the configuration class. For Swift, we might want to create the configuration class per ApiClient instead of singleton as other API clients (e.g. C#, PHP, Ruby) also follows the per-ApiClient design (instead of having all instances of ApiClient share the same configuration)

Swagger-codegen version

2.3.0

If anyone has any suggestion/feedback, or wants to work on this enhancement, please reply to let us know. Thank you.

Contributor guide

Research direction
Study the ObjC DefaultConfiguration class and Python's ApiClient configuration design to implement a per ApiClient configuration class for Swift.
Tech stack
swiftobjective c
Domain
apitooling
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-2 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
SwiftObjective C
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60