Closed Bug 903222 Opened 11 years ago Closed 11 years ago

Use HTTPS links when possible [PHP]

Categories

(www.mozilla.org :: Pages & Content, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: kohei, Assigned: kohei)

References

()

Details

(Keywords: sec-low)

Attachments

(16 files, 2 obsolete files)

1.12 MB, patch
Details | Diff | Splinter Review
462.97 KB, patch
Details | Diff | Splinter Review
950.29 KB, patch
Details | Diff | Splinter Review
417.44 KB, patch
Details | Diff | Splinter Review
463.77 KB, patch
Details | Diff | Splinter Review
527.28 KB, patch
Details | Diff | Splinter Review
345.58 KB, patch
Details | Diff | Splinter Review
35.42 KB, patch
Details | Diff | Splinter Review
192.40 KB, patch
Details | Diff | Splinter Review
93.15 KB, patch
Details | Diff | Splinter Review
112.67 KB, patch
Details | Diff | Splinter Review
43.71 KB, patch
Details | Diff | Splinter Review
349.89 KB, patch
Details | Diff | Splinter Review
313.44 KB, patch
Details | Diff | Splinter Review
53.36 KB, patch
Details | Diff | Splinter Review
437.15 KB, patch
Details | Diff | Splinter Review
+++ This bug was initially created as a clone of Bug #798611 +++

A spin-off from Bug 798611.

* Use HTTPS links when possible, both for Mozilla properties and external links
* Use absolute path for internal links

Patch coming.
Attached patch wiki.mozilla.orgSplinter Review
Attachment #788589 - Attachment description: www.mozilla.org rest of en-US → www.mozilla.org other
Attached patch www.google.*Splinter Review
Attached patch code.google.comSplinter Review
Attached patch docs.google.com (obsolete) — Splinter Review
Attached patch www.linkedin.com (obsolete) — Splinter Review
Comment on attachment 788612 [details] [diff] [review]
docs.google.com

Hmm, these links are on the old central page which is no longer used in major locales. Discard this patch.
Attachment #788612 - Attachment is obsolete: true
Comment on attachment 788613 [details] [diff] [review]
www.linkedin.com

Same here... the old central page.
Attachment #788613 - Attachment is obsolete: true
Whiteboard: [kb=1095150]
Kohei, is this it or are there more patches on the way?
That's all.
Comment on attachment 788614 [details] [diff] [review]
Facebook + Twitter

Review of attachment 788614 [details] [diff] [review]:
-----------------------------------------------------------------

::: af/firefox/4/firstrun/fallback.inc.html
@@ -18,5 @@
>    <div class="sub-feature" id="sidebar">
>      <h2>Bly in kontak</h2>
>      <ul class="link">
> -      <li id="connect-twitter"><a href="http://twitter.com/firefox">Volg ons op Twitter</a></li>
> -      <li id="connect-facebook"><a href="http://af-za.facebook.com/Firefox">Word ’n Facebook-ondersteuner</a></li>

Is it intenational that the facebook URL changed from af-za. to www. here?
Priority: -- → P3
(In reply to Steven Garrity [:sgarrity] from comment #23)
> Is it intenational that the facebook URL changed from af-za. to www. here?

It's not intended, but it should be okay as Facebook automatically detects user's language.
We talking about this in our weekly mozilla.org check-in meeting and given how many files are being removed from SVN, and the difficulty in carefully reviewing these large patches, we might leave this un-applied and hope to retire the PHP site instead.

If that takes too long, we could revisit with a smaller patch set when there are fewer pages left.
Yeah, we should focus on migrating to Bedrock instead. Closing as WONTFIX.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
Priority: P3 → --
Whiteboard: [kb=1095150]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: