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

Able to view certificate even after the URL is removed #9172

Closed
srirambv opened this issue May 31, 2017 · 1 comment
Closed

Able to view certificate even after the URL is removed #9172

srirambv opened this issue May 31, 2017 · 1 comment

Comments

@srirambv
Copy link
Collaborator

srirambv commented May 31, 2017

Test plan

#9378 (comment)


  • Did you search for similar issues before submitting this one?
    Yes

  • Describe the issue you encountered:
    Able to view certificate even after the URL is removed

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Windows 10 x64

  • Brave Version (revision SHA):
    Brave 0.15.312
    rev cc653ff
    Muon 3.0.202

  • Steps to reproduce:

    1. Visit any site in a new tab
    2. Click and press Ctrl+X to cut the URL
    3. Click on the magnififer icon, shows the connection info modal and still able to view the certificate
  • Actual result:
    Able to view certificate even after the URL is removed

  • Expected result:
    Once URL is removed connection info modal should not be shown when clicked on the magnifier glass

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    yes

  • Is this an issue in the currently released version?
    Not verified but should be an existing bug
    Verified existing issue on stable release build

  • Can this issue be consistently reproduced?
    Yes

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:
    connection info

  • Any related issues:
    cc: @bbondy @diracdeltas

@srirambv srirambv added this to the 0.17.x (Frozen, only critical adds from here) milestone May 31, 2017
@bsclifton
Copy link
Member

Good catch- this is not possible in Chrome

@cezaraugusto cezaraugusto self-assigned this Jun 9, 2017
cezaraugusto added a commit that referenced this issue Jun 10, 2017
Auditors: @diracdeltas
Close #9172
Test Plan:
covered by automated tests
@alexwykoff alexwykoff modified the milestones: 0.19.x, 0.17.x (Frozen, only critical adds from here) Jun 12, 2017
bridiver pushed a commit that referenced this issue Jun 14, 2017
Auditors: @diracdeltas
Close #9172
Test Plan:
covered by automated tests
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.