#1 Add host to domain transformation method

Злито
jonathan злито 1 комітів з kris/host_to_domain до master 4 роки тому
kris прокоментував(ла) 4 роки тому

@jonathan please review

@jonathan please review
jonathan був призначений kris 4 роки тому
jonathan зробив запит змін 4 роки тому
jonathan left a comment

Question about removal of getHost call to validate URI.

@@ -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 прокоментував(ла) 4 роки тому

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 прокоментував(ла) 4 роки тому

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 прокоментував(ла) 4 роки тому

Sorry, I missed that. Thanks.

Sorry, I missed that. Thanks.
jonathan closed this pull request 4 роки тому
jonathan видалена гілка kris/host_to_domain 4 роки тому

Рецензенти

jonathan зробив запит змін 4 роки тому
Запит на злиття був влитиий як 45579ec840.
Підпишіться щоб приєднатися до обговорення.
No reviewers
Без мітки
Етап відсутній
Немає виконавеця
2 учасників
Сповіщення
Дата завершення

Термін виконання не встановлений.

Залежності

Цей запит на злиття в даний час не має залежностей.

Завантаження…
Тут ще немає жодного змісту.