-
Notifications
You must be signed in to change notification settings - Fork 974
Fix displayed mixed content status #12743
Conversation
confirmed that the travis mixed content test fails on master (https://travis-ci.org/brave/browser-laptop/jobs/331029607#L6166) but passes with this PR |
// partially-secure connection. It can only downgrade a secure | ||
// connection. | ||
isSecure = | ||
e.securityInfo.mixedContentStatus === 'content-status-displayed' && this.props.isSecure !== false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we need to handle content-status-displayed-and-ran
which indicates both passive and active mixed content?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
never mind, I forgot we default block that active mixed content scripts so it won't reach that case
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, i tried allowing insecure scripts on https://very.badssl.com/, but it only emitted content-status-displayed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
++
// partially-secure connection. It can only downgrade a secure | ||
// connection. | ||
isSecure = | ||
e.securityInfo.mixedContentStatus === 'content-status-displayed' && this.props.isSecure !== false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
never mind, I forgot we default block that active mixed content scripts so it won't reach that case
Fix displayed mixed content status
Fix displayed mixed content status
Fix #12742
Test Plan:
Submitter Checklist:
git rebase -i
to squash commits (if needed).Test Plan:
Reviewer Checklist:
Tests