Skip to content

fix: upgrade site URLs from HTTP to HTTPS#2850

Open
JasonOA888 wants to merge 2 commits intosherlock-project:masterfrom
JasonOA888:fix/http-to-https-site-urls
Open

fix: upgrade site URLs from HTTP to HTTPS#2850
JasonOA888 wants to merge 2 commits intosherlock-project:masterfrom
JasonOA888:fix/http-to-https-site-urls

Conversation

@JasonOA888
Copy link
Copy Markdown

Upgrades 5 site configurations from plain HTTP to HTTPS.

All 5 sites confirmed to support HTTPS:

  • Gravatar (en.gravatar.com) → 302 redirect to HTTPS
  • PromoDJ (promodj.com) → 200 OK on HTTPS
  • Wikidot (www.wikidot.com) → 200 OK on HTTPS
  • authorSTREAM (www.authorstream.com) → 200 OK on HTTPS
  • datingRU (dating.ru) → 200 OK on HTTPS

Using plain HTTP leaks username queries on the wire and is vulnerable to MITM attacks. Two additional sites (igromania, uid.me) were left unchanged as their HTTPS endpoints were unreachable.

All 5 sites (Gravatar, PromoDJ, Wikidot, authorSTREAM, datingRU)
confirmed to support HTTPS. Using plain HTTP leaks query data and
is vulnerable to MITM attacks.

Verified HTTPS support for each site before updating.
@github-actions
Copy link
Copy Markdown
Contributor

Automatic validation of changes

Target F+ Check F- Check
Gravatar ✔️   Pass ❌   Fail
datingRU ✔️   Pass ✔️   Pass
authorSTREAM ✔️   Pass ❌   Fail
PromoDJ ✔️   Pass ✔️   Pass
Wikidot ✔️   Pass ✔️   Pass

Failures were detected on at least one updated target. Commits containing accuracy failures will often not be merged (unless a rationale is provided, such as false negatives due to regional differences).

Gravatar's en.gravatar.com redirects to gravatar.com which returns 404
for all users - detection is broken regardless of protocol. Keep HTTP
to avoid changing a broken target.

authorSTREAM failed F- check on HTTPS - reverting until investigated.
@github-actions
Copy link
Copy Markdown
Contributor

Automatic validation of changes

Target F+ Check F- Check
datingRU ✔️   Pass ✔️   Pass
Wikidot ✔️   Pass ✔️   Pass
PromoDJ ✔️   Pass ✔️   Pass

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