HomePhabricator

Fix URI escaping, which should actually be "%s", not "%p"
Audit Required45f421154179

Description

Fix URI escaping, which should actually be "%s", not "%p"

See https://discourse.phabricator-community.org/t/projects-workboards-links-issue/2896.

The documentation on urisprintf() isn't very clear here, I'll update it in
a followup. "%p" is for cases like encoding a branch name (which may contain
slashes) as a single path component in a URI.

Details

Group Auditors
Cowboy Commits
Provenance
epriestleyAuthored on Jul 4 2019, 6:00 PM
epriestleyPushed on Jul 4 2019, 6:01 PM
Parents
rP02315c4c48b9: Fix double-close on dialogs leading to Javascript console error
Branches
Unknown
Tags
Unknown
Build Status
Buildable 23111
Build 31731: Run Core Tests