spring-cloud/spring-cloud-netflix

Update Eureka client status URL based on SSL configuration

Open

#1843 aperta il 7 apr 2017

Vedi su GitHub
 (12 commenti) (0 reazioni) (0 assegnatari)Java (2433 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (4805 star)
Metriche merge PR
 (Merge medio 3h 47m) (5 PR mergiate in 30 g)

Descrizione

Hi,

If I use secure-port in my configuration and I don't specify the management.port, then spring cloud does not change my eureka statusPageUrl and other url's to https.
The if condition at line number 114 in EurekaClientAutConfiguraiton.java (spring-cloud-netflix-eureka-client: 1.2.6 release) prevents it from changing the scheme and it lets netflix configure the url . In netflix's InstanceInfo.java there is no check for secure port and the protocol is hardcoded to http.

I am not sure where to raise this issue, but since I am using spring-cloud-netflix I am putting this here

Guida contributor

Direzione di ricerca
Esamina la condizione alla riga 114 di EurekaClientAutoConfiguration.java per capire perché lo schema non viene aggiornato a HTTPS quando viene impostata la porta sicura e non viene specificata management.port. Inoltre, esamina InstanceInfo.java di Netflix per vedere come vengono costruite le URL con HTTP hardcoded.
Tech stack
java
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaSpring Cloud
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50