[wp-trac] [WordPress Trac] #54372: class-wp-automatic-updater.php adds a period to home_url()

[WordPress Trac]

#54372: class-wp-automatic-updater.php adds a period to home_url()
-----------------------------------------+------------------------------
 Reporter:  chris554                     |       Owner:  (none)
     Type:  defect (bug)                 |      Status:  new
 Priority:  normal                       |   Milestone:  Awaiting Review
Component:  Upgrade/Install              |     Version:  trunk
 Severity:  minor                        |  Resolution:
 Keywords:  reporter-feedback has-patch  |     Focuses:  administration
-----------------------------------------+------------------------------

Comment (by chris554):

 Hi @costdev, thank you for the amazingly quick response.

 The plugin I'm using for SMTP emails is "WP Mail SMTP", and within that
 plugin the only configuration is adding my API key to their built-in
 integration with SendGrid. My understanding is that WP Mail SMTP grabs all
 outgoing emails and sends them via API call to SendGrid, which handles
 mail delivery and link branding, etc.

 I had opened a ticket with SendGrid prior to this ticket, and they
 confirmed that on their end the JSON received from WP Mail SMTP contained
 the period as part of the URL, closing the ticket on their end. It only
 affected that one specific place where the home_url() is followed by a
 period, all the other links in the emails have spaces preceding and
 following.

 I think, personally, the 54372.earlier-location.diff is a cleaner fix.

 Still new to this process, is there anything I should do to test this,
 other than just changing the wp-admin/includes/class-wp-automatic-
 updater.php files on my site to the changed ones?

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/54372#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform