It would be good to default to SSL.

CommentFileSizeAuthor
#2 ssl-issue2911624-comment2.patch2.51 KBlaryn

Comments

laryn created an issue. See original summary.

laryn’s picture

laryn
he/him
commented
StatusFileSize
new ssl-issue2911624-comment2.patch2.51 KB

Patch attached.

adamps’s picture

adamps commented

Thanks. Did you test that they all still work?

laryn’s picture

laryn
he/him
commented

Yes, with the exception of Pinterest (as noted in the other thread -- where I'll follow up again) and VK which I have no way to test (no account there). But they both do respond on the https channel so I think it is safe. Here are some sample links with https in case you want to test simply:

- https://www.linkedin.com/shareArticle?mini=true&url=https://www.drupal.o...

- https://pinterest.com/pin/create%2Fbutton/?url=https://www.drupal.org/no...

- https://www.reddit.com/submit?url=https://www.drupal.org/node/2911624&ti...

- https://tumblr.com/share?s=&v=3&t=Default-to-SSL&u=https://www.drupal.or...

- https://npr.tumblr.com/

- https://vk.com/share.php?url=https://www.drupal.org/node/2911624

  • AdamPS committed fb8d99c on 8.x-2.x 
    Issue #2911624 by laryn: Default to SSL
adamps’s picture

adamps commented
Status: Active » Fixed
adamps’s picture

adamps commented
Assigned: laryn » Unassigned

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.