Enhancement : migrazione a java 11 o superiore

italia / spid-spring

SPID extension for Java Spring

BSD 3-Clause "New" or "Revised" License

29 stars 22 forks source link

Enhancement : migrazione a java 11 o superiore #16

Open FaberNa opened 3 years ago

FaberNa commented 3 years ago

Ciao, valutate il rilascio su una versione di java contemporanea ? Si potrebbe iniziare a lavorarci o ci sono vincoli per cui la jdk sia sempre la 8 ?

peppelinux commented 2 years ago

Ciao @FaberNa ho integrato diverse PR pendenti, tra le quali un tuo importante contributo. Se puoi proporre questo avanzamento con una PR te ne sarei grato, la unirei asap

FaberNa commented 2 years ago

Ok, lo propongo, ho notato alcuni test che falliscono su master ... Per via di restituire l'xml piuttosto che il contenuto codificato in base64

rdndnl commented 2 years ago

A mio modesto parere non vedo quale sia il vantaggio a rendere obbligatorio l'utilizzo di jdk 11+, soprattutto in un progetto che si appoggia ad un altro framework (spring-boot) che ha già i propri requisiti minimi (JDK 1.8+). Credo abbia senso restringere l'utilizzo soltanto se si introducono sostanziali vantaggi:

le linee di codice si riducono?
l'implementazione è più veloce?
l'applicazione finale importa meno librerie esterne?
l'applicazione sarà più sicura?

NOTA: Mi riferisco sempre alla fase di compilazione, perché in esecuzione deve funzionare sempre anche con java 11 o superiore.

FaberNa commented 2 years ago

@rdndnl per ambiente di esecuzione penso che concorderai che oramai ancora fino al 2023 sia la 11 la nuova LTS.
https://www.oracle.com/java/technologies/java-se-support-roadmap.html Comunque sia riprendendo la lista :

le linee di codice si riducono? --> Non vedo molte differenze
l'implementazione è più veloce? --> Impercettibile il miglioramento, anzi in alcuni test sono identici
l'applicazione finale importa meno librerie esterne? --> La dimensione è identica
l'applicazione sarà più sicura? --> su questo non saprei aggiungere altro

Non capisco perchè potresti vedere aspetti negativi in questo upgrade ?

N.B. la PR è anche abbastanza semplice .... sarei quasi tentato dal farla ...

peppelinux commented 2 years ago

Diciamo che è sempre meglio avanzare di pari passo le release con le versioni delle loro dipendenze.

Se avanzare da subito jdk non risulta un requisito necessario facciamo una release con jdk1.8 e la prossima con jdk11. Possibilmente renderla retrocompatibile, tentiamo di avere in CI tutte le jdk che riteniamo adottare.

Vi torna?