Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Regression: mixed content site showing up as insecure #5524

Closed
diracdeltas opened this issue Nov 10, 2016 · 0 comments
Closed

Regression: mixed content site showing up as insecure #5524

diracdeltas opened this issue Nov 10, 2016 · 0 comments
Assignees
@diracdeltas
Labels
QA/checked-Linux QA/checked-macOS QA/checked-Win32 QA/checked-Win64 security
Milestone
0.12.9dev

Comments

@diracdeltas
Copy link
Member

test plan:

  1. go to https://mixed.badssl.com/
  2. it should show up as secure, not insecure
@diracdeltas diracdeltas added this to the 0.12.9dev milestone Nov 10, 2016
@diracdeltas diracdeltas self-assigned this Nov 10, 2016
diracdeltas added a commit to brave/muon that referenced this issue Nov 11, 2016
@diracdeltas
Emit more specific security state change events
b6eb04e 
Needed for brave/browser-laptop#5524

Auditors: @bridiver @darkdh
@diracdeltas diracdeltas mentioned this issue Nov 11, 2016
Merged
diracdeltas added a commit that referenced this issue Nov 11, 2016
@diracdeltas
Show passive mixed content as secure
ec9ea64 
Requires brave/muon#92
Fix #5524

Auditors: @darkdh @bridiver

Test Plan: covered by automated test
@diracdeltas diracdeltas mentioned this issue Nov 11, 2016
Merged
4 tasks
This was referenced Nov 14, 2016
Closed
Closed
Closed
@luixxiul luixxiul mentioned this issue Nov 14, 2016
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@diracdeltas diracdeltas

Labels
QA/checked-Linux QA/checked-macOS QA/checked-Win32 QA/checked-Win64 security
Projects
None yet
Milestone
0.12.9dev
Development

No branches or pull requests
4 participants
@diracdeltas @alexwykoff @luixxiul @srirambv