Paths

Table of Contentst

Differential D21009

Whitelist "vscode://" as an allowed Editor protocol
ClosedPublic
Actions

Authored by epriestley on Feb 20 2020, 8:41 PM.

Tags

None

Referenced Files

	F18734162: D21009.id.diff
	Tue, Sep 30, 10:49 PM

	F18732408: D21009.diff
	Tue, Sep 30, 6:04 PM

	F18653562: D21009.id50053.diff
	Sun, Sep 21, 2:20 PM

	F18649548: D21009.diff
	Sat, Sep 20, 9:19 PM

	F18638964: D21009.id.diff
	Sep 17 2025, 11:58 AM

	F18635969: D21009.diff
	Sep 17 2025, 3:06 AM

	F18630841: D21009.id.diff
	Sep 16 2025, 1:21 PM

	F18630787: D21009.diff
	Sep 16 2025, 12:56 PM

Subscribers

None

Details

Reviewers: None
Commits: rP088dfdb586df: (stable) Whitelist "vscode://" as an allowed Editor protocol
rP149155ee20e8: Whitelist "vscode://" as an allowed Editor protocol

Summary

See PHI1647, which asks for "vscode://" to be a configurable protocol on hosted Phacility instances.

I made the configuration editable in D21008, but this can reasonably just come upstream too.

Test Plan

Viewed config in Config, set my editor URI to vscode://blahblah.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 20 2020, 8:41 PM

Harbormaster completed remote builds in B23875: Diff 50051.Feb 20 2020, 8:43 PM

epriestley requested review of this revision.Feb 20 2020, 8:43 PM

This revision was not accepted when it landed; it landed in state Needs Review.Feb 20 2020, 8:45 PM

Closed by commit rP149155ee20e8: Whitelist "vscode://" as an allowed Editor protocol (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP149155ee20e8: Whitelist "vscode://" as an allowed Editor protocol.

epriestley added a commit: rP088dfdb586df: (stable) Whitelist "vscode://" as an allowed Editor protocol.Apr 5 2020, 4:47 PM

Revision Contents
Changeset List

Path

Size

src/

applications/

config/

option/

PhabricatorSecurityConfigOptions.php

3 lines

Diff 50053

src/applications/config/option/PhabricatorSecurityConfigOptions.php

Loading...