Add host to domain transformation method

Yeni Değişiklik İsteği

Sohbet 0 İşlemeler 1 Değiştirilen Dosyalar 1

@jonathan please review

jonathan 4 yıl önce kris tarafından atandı

kris bu değişiklik isteğine referansta bulundu bubblev/bubble'den 4 yıl önce

First part of fixing backup-restore test #12

jonathan 4 yıl önce değişiklik istedi

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 4 yıl önce yorum yaptı

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 yıl önce yorum yaptı

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 yıl önce yorum yaptı

Sorry, I missed that. Thanks.

jonathan 4 yıl önce işlemesinde bu konuyu işaret etti

Add host to domain transformation method (#1) Add host to domain transformation method Co-authored-by: Kristijan Mitrovic <kmitrovic@itekako.com> Reviewed-on: https://git.bubblev.org/bubblev/cobbzilla-utils/pulls/1

jonathan 4 yıl önce değişiklik isteğini kapattı

jonathan kris/host_to_domain dalı silindi 4 yıl önce

Gözden Geçirenler

jonathan 4 yıl önce değişiklik istedi

Değişiklik isteği 45579ec840 olarak birleştirildi.

Bu konuşmaya katılmak için oturum aç.

jonathan

Etiket Yok

Kilometre Taşı Yok

jonathan

2 Katılımcı

Bildirimler

Bitiş Tarihi

Bitiş tarihi atanmadı.

Bağımlılıklar

Bu çekme isteği henüz bir bağımlılık içermiyor.

#1 Add host to domain transformation method

Gözden Geçirenler