Page MenuHomePhabricator

Add `urisprintf()` for encoded printing of URIs
ClosedPublic

Authored by epriestley on Nov 9 2013, 2:31 AM.
Tags
None
Referenced Files
F18737815: D7538.id.diff
Wed, Oct 1, 12:46 PM
F18725113: D7538.diff
Tue, Sep 30, 4:46 AM
F18624674: D7538.diff
Mon, Sep 15, 8:45 PM
F18512956: D7538.diff
Fri, Sep 5, 8:46 AM
F18437496: D7538.id.diff
Aug 31 2025, 12:10 PM
F18410073: D7538.diff
Aug 30 2025, 4:38 AM
F18087078: D7538.id17014.diff
Aug 6 2025, 2:13 AM
F17940453: D7538.id17015.diff
Jul 31 2025, 4:24 AM
Subscribers

Details

Summary

Ref T1049. This is primarily to enable us to do a reasonable job of variable replacement in "Request URI" Harbormaster build steps. For example, the user will enter:

http://jenkins.mycompany.com/?diff=${buildable.name}

...which we'll transform to:

http://jenkins.mycompany.com/?diff=%s

...and then build correctly.

Test Plan

Unit tests.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Unknown Object (????).Nov 9 2013, 2:40 AM
  • Add a v version too, so the implementation can mirror the RemoteCommand one more closely.