Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Link protection prompts for github.com although it is in trusted domain #79898

Closed
stevencl opened this issue Aug 27, 2019 · 4 comments
Closed
Assignees
Labels
*duplicate Issue identified as a duplicate of another issue(s) workbench-link Link protection in workbench

Comments

@stevencl
Copy link
Member

stevencl commented Aug 27, 2019

Issue Type: Bug
Testing: #79465
Open release notes
Click August Iteration Plan link (#78844)
Get prompted if i want to open this external website
Click 'Configure Trusted Domains'
See that https://github.com is already checked
Expected: When clicking the August Iteration Plan link, the link should just open since it is marked as trusted

VS Code version: Code - Insiders 1.38.0-insider (8cd00a7, 2019-08-27T06:13:54.903Z)
OS version: Windows_NT x64 10.0.18362

System Info
Item Value
CPUs Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz (8 x 2112)
GPU Status 2d_canvas: enabled
flash_3d: enabled
flash_stage3d: enabled
flash_stage3d_baseline: enabled
gpu_compositing: enabled
multiple_raster_threads: enabled_on
native_gpu_memory_buffers: disabled_software
oop_rasterization: disabled_off
protected_video_decode: enabled
rasterization: enabled
skia_deferred_display_list: disabled_off
skia_renderer: disabled_off
surface_synchronization: enabled_on
video_decode: enabled
viz_display_compositor: disabled_off
webgl: enabled
webgl2: enabled
Load (avg) undefined
Memory (System) 15.93GB (2.34GB free)
Process Argv
Screen Reader yes
VM 0%
Extensions (12)
Extension Author (truncated) Version
tslint eg2 1.0.44
vscode-github-releases-updater joa 0.17.0
vscode-docker ms- 0.7.0
python ms- 2019.8.30787
remote-containers ms- 0.69.0
remote-ssh ms- 0.45.6
remote-ssh-edit ms- 0.45.6
remote-ssh-explorer ms- 0.45.6
remote-wsl ms- 0.39.4
vscode-remote-extensionpack ms- 0.16.0
azure-account ms- 0.8.4
test-search-rg rob 0.0.3
@stevencl
Copy link
Member Author

The issue is that it wasn't obvious when the configure trusted domains widget came up that the user has to press OK. On first launch there should be a prompt to tell the user to click ok to accept the defaults. Or by default nothing should be checked which would make it more obvious that the user can check the ones that they want to trust and then click OK.

@octref octref added polish Cleanup and polish issue under-discussion Issue is under discussion for relevance, priority, approach labels Aug 27, 2019
@octref
Copy link
Contributor

octref commented Aug 27, 2019

We do have some details text that explains what the item does:

image

Or by default nothing should be checked

I think if user is not interested in trusting the domain, he would have clicked the default action "Open Link", so I feel this should be opt-out.

@octref octref modified the milestones: August 2019, September 2019 Aug 27, 2019
@octref octref added *duplicate Issue identified as a duplicate of another issue(s) and removed polish Cleanup and polish issue under-discussion Issue is under discussion for relevance, priority, approach labels Sep 9, 2019
@vscodebot
Copy link

vscodebot bot commented Sep 9, 2019

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for existing issues here. See also our issue reporting guidelines.

Happy Coding!

@vscodebot vscodebot bot closed this as completed Sep 9, 2019
@octref
Copy link
Contributor

octref commented Sep 9, 2019

Merged to #80489.

@vscodebot vscodebot bot locked and limited conversation to collaborators Oct 24, 2019
@octref octref added the workbench-link Link protection in workbench label Oct 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*duplicate Issue identified as a duplicate of another issue(s) workbench-link Link protection in workbench
Projects
None yet
Development

No branches or pull requests

2 participants