spring-cloud/spring-cloud-netflix

Update Eureka client status URL based on SSL configuration

Aberta

#1.843 aberto em 7 de abr. de 2017

Ver no GitHubGuia do colaborador
 (12 comentários) (0 reação) (0 responsável)Java (2.433 forks)batch import
enhancementhelp wanted

Métricas do repositório

Stars
 (4.805 estrelas)
Métricas de merge de PR
 (Mesclagem média 3h 47m) (5 fundiu PRs em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
Examine a condição na linha 114 do EurekaClientAutoConfiguration.java para entender por que o esquema não é atualizado para HTTPS quando a porta segura está definida e a management.port não é especificada. Além disso, revise o InstanceInfo.java da Netflix para ver como as URLs são construídas com HTTP codificado.
Pilha de tecnologia
java
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaSpring Cloud
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50