spring-attic / spring-social-twitter

Twitter API binding and connect support.
http://projects.spring.io/spring-social-twitter
Apache License 2.0
122 stars 115 forks source link

URL Cleanup #136

Closed spring-operator closed 5 years ago

spring-operator commented 5 years ago

This commit updates URLs to prefer the https protocol. Redirects are not followed to avoid accidentally expanding intentionally shortened URLs (i.e. if using a URL shortener).

HTTP URLs that Could Not Be Fixed

These URLs were unable to be fixed. Please review them to see if they can be manually resolved.

Fixed URLs

Fixed But Review Recommended

These URLs were fixed, but the https status was not OK. However, the https status was the same as the http request or http redirected to an https URL, so they were migrated. Your review is recommended.

Fixed Success

These URLs were switched to an https URL with a 2xx status. While the status was successful, your review is still recommended.

Ignored

These URLs were intentionally ignored.

habuma commented 5 years ago

Many of the "needs review" items are in tests. At least one of the fixes actually changes the URL completely and, as such, is causing some tests to break, which is why Travis is complaining. I'm going to merge this PR, but then followup with a fix for the broken test.