Proposta evolutiva per creazione configurazione

[abrigantiWGS]

La libreria dipende da un file di configurazione JSON, sarebbe però bello poter creare la configurazione direttamente da codice (magari perché i parametri arrivano da altri fonti, esempio variabili di ambiente).

In fondo basterebbe far si che la configurazione abbia le proprietà definite e un costruttore diverso (o avere più classi config con una base comune.).

[francesco-loreti]

La libreria è progettata per consentire l’accesso a più API PDND e a più ambienti, utilizzando i parametri di configurazione che vengono passati dall’esterno.

Per questo motivo sarebbe limitante modificare il client affinché legga i valori da un’unica sorgente (ad esempio variabili d’ambiente locali o altri parametri specifici dell’applicativo).

A mio avviso è più corretto che sia l’applicativo a costruire il proprio dizionario di configurazione, senza dipendere dall’importazione di un file JSON.

Ad esempio, invece di:
config = Config("./configs/sample.json")

puoi creare dinamicamente la configurazione così

conf = {
    "kid": os.getenv("TUO_ENV_PER_KID"),
    "issuer": os.getenv("TUO_ENV_PER_ISSUER"),
    "clientId": os.getenv("TUO_ENV_PER_CLIENTID"),
    "purposeId": os.getenv("TUO_ENV_PER_PURPOSEID"),
    "privKeyPath": os.getenv("TUO_ENV_PER_KEY"),
}

e poi passare il dizionario direttamente al client:
jwt_gen = JWTGenerator(config)

In questo modo la libreria rimane flessibile e indipendente dalla sorgente dei parametri, mentre l’applicativo può decidere liberamente come costruire la configurazione.

Che ne pensi.