Page MenuHomePhabricator

urlutil: provide some information about "bad url" when processing `pushurl`
ClosedPublic

Authored by marmoute on Apr 15 2021, 8:56 PM.

Details

Summary

It appears pushurl only support <proto>:// entries. This is not obvious and
can lead to obscure error. We make the error less obscure as a start..

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

marmoute created this revision.Apr 15 2021, 8:56 PM
baymax updated this revision to Diff 27024.Apr 17 2021, 7:45 PM

✅ refresh by Heptapod after a successful CI run (🐙 💚)

Alphare accepted this revision.Thu, Apr 29, 9:46 AM
This revision is now accepted and ready to land.Thu, Apr 29, 9:46 AM
baymax updated this revision to Diff 27215.Thu, Apr 29, 1:47 PM

✅ refresh by Heptapod after a successful CI run (🐙 💚)

baymax updated this revision to Diff 27243.Fri, Apr 30, 8:26 AM

✅ refresh by Heptapod after a successful CI run (🐙 💚)

baymax updated this revision to Diff 27427.Mon, May 3, 9:53 AM

✅ refresh by Heptapod after a successful CI run (🐙 💚)

pulkit accepted this revision.Sat, May 15, 3:20 PM
baymax updated this revision to Diff 27962.Mon, May 17, 2:46 AM

✅ refresh by Heptapod after a successful CI run (🐙 💚)

baymax updated this revision to Diff 27965.Mon, May 17, 4:21 AM

✅ refresh by Heptapod after a successful CI run (🐙 💚)