bubblev
/
cobbzilla-utils
6
0
Fork 0
Código Issues 0 Pull requests 0 Versões 1 Wiki Atividade
Etiquetas Marcos
Novo pull request

#1 Add host to domain transformation method

Merge aplicado
jonathan aplicou merge dos 1 commits de kris/host_to_domain em master 4 anos atrás
Conversação 0 Commits 1 Arquivos alterados 1
kris comentou 4 anos atrás

@jonathan please review

@jonathan please review
jonathan foi atribuído por kris 4 anos atrás
jonathan alterações solicitadas 4 anos atrás
jonathan left a comment

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 comentou 4 anos atrás

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 comentou 4 anos atrás

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 comentou 4 anos atrás

Sorry, I missed that. Thanks.

Sorry, I missed that. Thanks.
jonathan closed this pull request 4 anos atrás
jonathan excluiu branch kris/host_to_domain 4 anos atrás

Revisores

jonathan alterações solicitadas 4 anos atrás
O pull request teve merge aplicado como 45579ec840.
Acesse para participar desta conversação.
Revisores
Request review
No reviewers
 jonathan
Etiquetas
Apply labels
Limpar etiquetas
No items
Sem etiqueta
Marco
Set milestone
Limpar marco
No items
Sem marco
Responsáveis
Assign users
Limpar responsáveis
Sem responsável
 jonathan
2 participante(s)
Notificações
Data limite

Data limite não informada.

Dependências

Atualmente este pull request não tem dependências.

Escrever Pré-visualização
Carregando…
Cancelar
Salvar
Ainda não há conteúdo.
Excluir branch 'kris/host_to_domain'

A exclusão de uma branch é permanente. Isto NÃO PODERÁ ser desfeito. Continuar?

Não
Sim