bubblev
/
cobbzilla-utils
6
0
Derivar 0
Código Questões 0 Pedidos de integração 0 Lançamentos 1 Wiki Trabalho
Etiquetas Etapas
Novo pedido de integração

#1 Add host to domain transformation method

Integração executada
jonathan integrou 1 cometimentos de %!<(string=kris/host_to_domain)/code> em master há 4 anos
Diálogo 0 Cometimentos 1 Ficheiros modificados 1
kris comentado há 4 anos

@jonathan please review

@jonathan please review
jonathan foi atribuída por kris há 4 anos
jonathan alterações solicitadas há 4 anos
jonathan deixou um comentário

Question about removal of getHost call to validate URI.

src/main/java/org/cobbzilla/util/http/URIUtil.java
@@ -53,4 +61,2 @@
* @return the "registered" domain, which includes the TLD and one level up.
*/
public static String getRegisteredDomain(String uri) {
final String host = getHost(uri);
jonathan comentado há 4 anos

The getHost call was used to ensure that the uri argument is a valid URI. Can we leave it in? If that breaks something, maybe use a different method?

The `getHost` call was used to ensure that the `uri` argument is a valid URI. Can we leave it in? If that breaks something, maybe use a different method?
kris comentado há 4 anos

Nothing is removed. getRegisteredDomain still works as before - redefined in line 56 above, and does call getHost.

hostToDomain is a brand new method with the second part of the previous method required and used currently only in some test.

Nothing is removed. `getRegisteredDomain` still works as before - redefined in line 56 above, and does call `getHost`. `hostToDomain` is a brand new method with the second part of the previous method required and used currently only in some test.
jonathan comentado há 4 anos

Sorry, I missed that. Thanks.

Sorry, I missed that. Thanks.
jonathan fechou este pedido de integração há 4 anos
jonathan eliminou o ramo kris/host_to_domain há 4 anos

Revisores

jonathan alterações solicitadas há 4 anos
A integração constante no pedido foi executada como 45579ec840.
Inicie a sessão para participar neste diálogo.
Revisores
Solicitar avaliação
Sem revisores
 jonathan
Etiquetas
Aplicar etiquetas
Limpar etiquetas
Sem itens
Sem etiquetas
Etapa
Definir etapa
Limpar etapa
Sem itens
Sem etapa
Responsáveis
Definir responsáveis
Limpar responsáveis
Sem responsáveis
 jonathan
2 Participantes
Notificações
Date limite

Sem data limite definida.

Dependências

Este pedido de integração não tem quaisquer dependências, neste momento.

Escrever Pré-visualizar
Carregando…
Cancelar
Guardar
Ainda não há conteúdo.
Eliminar o ramo 'kris/host_to_domain'

Eliminar um ramo é algo permanente. NÃO PODERÁ ser revertido. Quer continuar?

Não
Sim