spring-cloud/spring-cloud-netflix

Update Eureka client status URL based on SSL configuration

Offen

#1.843 geöffnet am 07.04.2017

Auf GitHub ansehenContributor Guide
 (12 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Java (2.433 Forks)batch import
enhancementhelp wanted

Repository-Metriken

Stars
 (4.805 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche die Bedingung in Zeile 114 in EurekaClientAutoConfiguration.java, um zu verstehen, warum das Schema nicht auf HTTPS aktualisiert wird, wenn der sichere Port gesetzt ist und management.port nicht angegeben wird. Überprüfe auch Netflix' InstanceInfo.java, um zu sehen, wie URLs mit hartcodiertem HTTP erstellt werden.
Tech Stack
java
Domain
backend
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
JavaSpring Cloud
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50