879020 - "Enable SSL for our sites, and test for mixed content issues"

Status: RESOLVED FIXED

(Webmaker Graveyard :: DevOps, defect)

Description

[Brett Gaylor [:brett]]

In our conversation, this was deemed the resolution to mixed content warnings in our make details pages.

Comment 1

[Jon Buckley [:jbuck]]

Attachment: https://github.com/mozilla/login.webmaker.org/pull/80

First in a series of patches to enable SSL across our apps.

Comment 2

[Chris DeCairos (:cade)]

Comment on attachment 758087 [details] [review]
https://github.com/mozilla/login.webmaker.org/pull/80

R+ nothing looks out of the ordinary.

Comment 3

[[github robot]]

Commit pushed to master at https://github.com/mozilla/login.webmaker.org

https://github.com/mozilla/login.webmaker.org/commit/467395c9819dd667ca0707edc8605b5b4b4f6def
Bug 879020 - Add option to force SSL for all requests

Comment 4

[Jon Buckley [:jbuck]]

Attachment: https://github.com/mozilla/MakeAPI/pull/74

Comment 5

[Chris DeCairos (:cade)]

Comment on attachment 758113 [details] [review]
https://github.com/mozilla/MakeAPI/pull/74

r++

Comment 6

[[github robot]]

Commit pushed to master at https://github.com/mozilla/MakeAPI

https://github.com/mozilla/MakeAPI/commit/18fa04f87c6c4f6cd297939870f33b2b84393a4f
Bug 879020 - Add option to force SSL for all requests

Comment 7

[Jon Buckley [:jbuck]]

Attachment: https://github.com/mozilla/webmaker.org/pull/75

Comment 8

[Chris DeCairos (:cade)]

Comment on attachment 758122 [details] [review]
https://github.com/mozilla/webmaker.org/pull/75

R+

Comment 9

[[github robot]]

Commit pushed to master at https://github.com/mozilla/webmaker.org

https://github.com/mozilla/webmaker.org/commit/cb591346c498b6d8a793cf924db25d80f982fb64
Bug 879020 - Add option to force SSL for all requests

Comment 10

[Jon Buckley [:jbuck]]

Attachment: https://github.com/mozilla/thimble.webmaker.org/pull/94

Comment 11

[Jon Buckley [:jbuck]]

Attachment: https://github.com/mozilla/popcorn.webmaker.org/pull/33

Comment 12

[Chris DeCairos (:cade)]

Comment on attachment 758154 [details] [review]
https://github.com/mozilla/thimble.webmaker.org/pull/94

iz good.

Comment 13

[Chris DeCairos (:cade)]

Comment on attachment 758167 [details] [review]
https://github.com/mozilla/popcorn.webmaker.org/pull/33

o/

Comment 14

[[github robot]]

Commit pushed to master at https://github.com/mozilla/thimble.webmaker.org

https://github.com/mozilla/thimble.webmaker.org/commit/33bff2e1f75254a3d4b17c5ed7d32a814ebfd879
Bug 879020 - Add option to force SSL for all requests

Comment 15

[[github robot]]

Commit pushed to master at https://github.com/mozilla/popcorn.webmaker.org

https://github.com/mozilla/popcorn.webmaker.org/commit/d8a457a5d4b01c057956e293e90fb283d750057b
Bug 879020 - Add option to force SSL for all requests

Comment 16

[Jon Buckley [:jbuck]]

Okay, all the code is pushed. I will test enabling SSL tomorrow!

Comment 17

[Jon Buckley [:jbuck]]

SSL has been enabled for login and makeapi. Once any problems with those are fleshed out, I will enable it for webmaker, thimble, and butter.

Comment 18

[Jon Buckley [:jbuck]]

Alright, the switch is flipped for all our apps. Now it will be easier for people to find mixed content issues!