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

Update redirected URLs from https -> http (part 18) #15267

Merged
merged 2 commits into from
Apr 24, 2022

Conversation

OnkarRuikar
Copy link
Contributor

Extending #14055

Summary

Updated redirected URLs. Redirecting from https to http.

Metadata

  • Fixes a typo, bug, or other error

@OnkarRuikar OnkarRuikar requested a review from a team April 23, 2022 02:16
@OnkarRuikar OnkarRuikar requested review from a team as code owners April 23, 2022 02:16
@OnkarRuikar OnkarRuikar requested review from wbamberg, teoli2003 and dipikabh and removed request for a team April 23, 2022 02:16
@github-actions github-actions bot added Content:Accessibility Accessibility docs Content:HTML Hypertext Markup Language docs Content:HTTP HTTP docs Content:Learn Learning area docs Content:Other Any docs not covered by another "Content:" label Content:SVG SVG docs Content:WebAPI Web API docs labels Apr 23, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Apr 23, 2022

Preview URLs

Flaws

Note! 7 documents with no flaws that don't need to be listed. 🎉

URL: /en-US/docs/Web/API/console
Title: console
on GitHub
Flaw count: 1

  • macros:
    • /en-US/docs/Web/API/Console/exception does not exist

URL: /en-US/docs/Games/Techniques/3D_on_the_web
Title: 3D games on the Web
on GitHub
Flaw count: 1

  • broken_links:
    • Can't resolve /en-US/docs/Games/Techniques/3D_on_the_web/Building_up_a_basic_demo_with_Whitestorm.js

URL: /en-US/docs/Games/Tutorials/HTML5_Gamedev_Phaser_Device_Orientation
Title: 2D maze game with device orientation
on GitHub
Flaw count: 2

  • broken_links:
    • Can't resolve /en-US/docs/Web/Apps/Fundamentals/gather_and_modify_data/responding_to_device_orientation_changes
    • Can't resolve /en-US/docs/Web/Apps/Fundamentals/gather_and_modify_data/responding_to_device_orientation_changes

URL: /en-US/docs/Mozilla/Firefox/Releases/1.5/What_s_new_in_1.5_alpha
Title: What's New in Deer Park Alpha
on GitHub
Flaw count: 3

  • broken_links:
    • Can't resolve /en-US/Drawing_Graphics_with_Canvas
    • Can't resolve https://developer.mozilla.org/en-US/docs/Archive/Web/XForms
    • Can't resolve /en-US/Using_the_Stylesheet_Service

URL: /en-US/docs/Learn/Getting_started_with_the_web/Installing_basic_software
Title: Installing basic software
on GitHub
Flaw count: 1

  • broken_links:
    • Link points to the page it's already on

External URLs

URL: /en-US/docs/Web/SVG
Title: SVG: Scalable Vector Graphics
on GitHub


URL: /en-US/docs/Web/API/console
Title: console
on GitHub


URL: /en-US/docs/Web/Accessibility/Understanding_Colors_and_Luminance
Title: Web Accessibility: Understanding Colors and Luminance
on GitHub


URL: /en-US/docs/Web/HTTP/Link_prefetching_FAQ
Title: Link prefetching FAQ
on GitHub


URL: /en-US/docs/Web/HTML/Element/wbr
Title: <wbr>: The Line Break Opportunity element
on GitHub


URL: /en-US/docs/Web/Guide/Audio_and_video_delivery/Cross-browser_audio_basics
Title: Cross-browser audio basics
on GitHub


URL: /en-US/docs/Games/Techniques/3D_on_the_web
Title: 3D games on the Web
on GitHub


URL: /en-US/docs/Games/Tutorials/2D_breakout_game_Phaser/Randomizing_gameplay
Title: Randomizing gameplay
on GitHub


URL: /en-US/docs/Games/Tutorials/HTML5_Gamedev_Phaser_Device_Orientation
Title: 2D maze game with device orientation
on GitHub


URL: /en-US/docs/Mozilla/Firefox/Releases/1.5/What_s_new_in_1.5_alpha
Title: What's New in Deer Park Alpha
on GitHub


URL: /en-US/docs/MDN/Guidelines/Video
Title: Video content on MDN
on GitHub

No new external URLs


URL: /en-US/docs/Learn/Getting_started_with_the_web/Installing_basic_software
Title: Installing basic software
on GitHub

(this comment was updated 2022-04-23 06:04:29.147109)

Copy link
Contributor

@teoli2003 teoli2003 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So this one is trickier:

  • We prefer not to have redirects
  • But we don't like http links.
    So we want to have https links that doesn't redirect.
    In case there aren't any, we prefer having an https link that redirect to an http link because, when they will fix their website to support https correctly, the link will be automatically fixed while the http one will not.

Co-authored-by: Jean-Yves Perrier <[email protected]>
@OnkarRuikar
Copy link
Contributor Author

In case there aren't any, we prefer having an https link that redirect to an http link because, when they will fix their website to support https correctly, the link will be automatically fixed while the http one will not.

I see. Mostly they go from https to http because they don't want to pay for the certificate anymore. This way when they renew the certificate the URL will stop redirecting.

Counter argument can be made that, taking users from https to http on same domain is considered deceptive. It'll look bad for those domains.

@teoli2003
Copy link
Contributor

I see. Mostly they go from https to http because they don't want to pay for the certificate anymore. This way when they renew the certificate the URL will stop redirecting.

There are free certificates nowadays. (See https://letsencrypt.org/ )

Copy link
Contributor

@teoli2003 teoli2003 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving, but waiting for Eric's review before merging.

@teoli2003
Copy link
Contributor

I'm changing strategy here: I will merge this to prevent the rotting of the bug. If Eric finds any problem, we'll fix it in a follow-up.

@teoli2003 teoli2003 merged commit 0373149 into mdn:main Apr 24, 2022
@OnkarRuikar OnkarRuikar deleted the onk-rhttps branch April 24, 2022 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:Accessibility Accessibility docs Content:HTML Hypertext Markup Language docs Content:HTTP HTTP docs Content:Learn Learning area docs Content:Other Any docs not covered by another "Content:" label Content:SVG SVG docs Content:WebAPI Web API docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants