Add host to domain transformation method

Jauns izmaiņu pieprasījums

Saruna 0 Revīzijas 1 Izmainītie faili 1

@jonathan please review

jonathan tika piešķirta problēma no kris pirms 4 gadiem

kris atsaucās uz šo izmaiņu pieprasījumu no bubblev/bubble pirms 4 gadiem

First part of fixing backup-restore test #12

jonathan pieprasīja izmaiņas pirms 4 gadiem

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 komentēja pirms 4 gadiem

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 komentēja pirms 4 gadiem

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 komentēja pirms 4 gadiem

Sorry, I missed that. Thanks.

jonathan pieminēja šo problēmu revīzijā pirms 4 gadiem

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 aizvēra šo izmaiņu pieprasījumu pirms 4 gadiem

jonathan izdzēsa atzaru kris/host_to_domain pirms 4 gadiem

Recenzenti

jonathan pieprasīja izmaiņas pirms 4 gadiem

Izmaiņu pieprasījums tika sapludināts ar revīziju 45579ec840.

Pierakstieties, lai pievienotos šai sarunai.

jonathan

Nav etiķešu

Nav atskaites punktu

jonathan

2 dalībnieki

Paziņojumi

Izpildes termiņš

Izpildes termiņš nav uzstādīts.

Atkarības

Šim izmaiņu pieprasījumam pagaidām nav nevienas atkarības.

#1 Add host to domain transformation method

Recenzenti