Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

URL Cleanup #163

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

spring-operator
Copy link

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.

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.

* http://ogp.me/ns (301) with 1 occurrences could not be migrated:
   ([https](https://ogp.me/ns) result SSLHandshakeException).

# 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.

* http://www.ultraq.net.nz/thymeleaf/layout (302) with 15 occurrences migrated to:
  https://github.com/ultraq/thymeleaf-layout-dialect ([https](https://www.ultraq.net.nz/thymeleaf/layout) result ConnectTimeoutException).
* http://api.github.com:1234/ (ConnectTimeoutException) with 1 occurrences migrated to:
  https://api.github.com:1234/ ([https](https://api.github.com:1234/) result ConnectTimeoutException).
* http://github.com:1234/ (ConnectTimeoutException) with 1 occurrences migrated to:
  https://github.com:1234/ ([https](https://github.com:1234/) result ConnectTimeoutException).
* http://ogp.me/ns/fb (302) with 1 occurrences migrated to:
  https://graph.facebook.com/schema/og/ ([https](https://ogp.me/ns/fb) result 400).
* http://ogp.me/ns/article (302) with 1 occurrences migrated to:
  https://graph.facebook.com/schema/og/article ([https](https://ogp.me/ns/article) result 400).
* http://ogp.me/ns/object (302) with 1 occurrences migrated to:
  https://graph.facebook.com/schema/og/object ([https](https://ogp.me/ns/object) result 400).
* http://ogp.me/ns/profile (302) with 1 occurrences migrated to:
  https://graph.facebook.com/schema/og/profile ([https](https://ogp.me/ns/profile) result 400).
* http://example.com/webhook (404) with 3 occurrences migrated to:
  https://example.com/webhook ([https](https://example.com/webhook) result 404).
* http://github.com/github/gollum/issues/1 (301) with 2 occurrences migrated to:
  https://github.com/github/gollum/issues/1 ([https](https://github.com/github/gollum/issues/1) result 404).
* http://projects.spring.io/spring-nopermission (404) with 1 occurrences migrated to:
  https://projects.spring.io/spring-nopermission ([https](https://projects.spring.io/spring-nopermission) result 404).

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

* http://github.com/github/linguist/issues/1 with 2 occurrences migrated to:
  github-linguist/linguist#1 ([https](github-linguist/linguist#1) result 200).
* http://spring.io/team/rwinch with 2 occurrences migrated to:
  https://spring.io/team/rwinch ([https](https://spring.io/team/rwinch) result 200).
* http://www.thymeleaf.org with 15 occurrences migrated to:
  https://www.thymeleaf.org ([https](https://www.thymeleaf.org) result 200).
* http://projects.spring.io/spring-framework with 1 occurrences migrated to:
  https://projects.spring.io/spring-framework ([https](https://projects.spring.io/spring-framework) result 301).
* http://spring.io/spring-security with 1 occurrences migrated to:
  https://spring.io/spring-security ([https](https://spring.io/spring-security) result 302).

# Ignored
These URLs were intentionally ignored.

* http://localhost with 1 occurrences
* http://localhost/github/hooks/pull_request/ with 1 occurrences
* http://localhost/login/oauth2/github with 2 occurrences
* http://localhost/sign/ with 7 occurrences
* http://localhost/sign/pivotal?repositoryId=rwinch/176_test&pullRequestId=2 with 5 occurrences
* http://localhost:8080/ with 1 occurrences
* http://localhost:8080/login/oauth2/github with 1 occurrences
@rwinch rwinch force-pushed the main branch 4 times, most recently from 01f616a to a3b7491 Compare May 1, 2024 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant