MS function get_blogaddress_by_domain returns output escaped by esc_url() - should be esc_url_raw()

[tlovett1]

Title is pretty self-explanatory. get_blogaddress_by_domain() runs esc_url() on it's output. get_blogaddress_by_domain() is only called once in core:

In site-info.php:

$blog_address = get_blogaddress_by_domain( $_POST['blog']['domain'], $_POST['blog']['path'] );
  if ( get_option( 'siteurl' ) != $blog_address )
    update_option( 'siteurl', $blog_address );

Since get_blogaddress_by_domain() is saved straight to the options table, it should be using the correct sanitization function, esc_url_raw().

[tlovett1]

Use esc_url_raw instead of esc_url

[SergeyBiryukov]

get_blogaddress_by_domain() should probably be deprecated once #18242 is resolved, see ticket:18242:14.

[nacin]

Even with #18242, I'm going to commit 25057.diff​, because it's definitely correct.

[nacin]

In 25033:

Return esc_url_raw(), not esc_url(), from get_blogaddress_by_domain(). props tlovett1, fixes #25057. #wcpvd