D2834: wireproto: support /api/* URL space for exposing APIs

indygreg (Gregory Szorc) phabricator at mercurial-scm.org
Tue Mar 13 11:46:42 EDT 2018


indygreg added a comment.


  In https://phab.mercurial-scm.org/D2834#45666, @mharbison72 wrote:
  
  > Will this cause conflicts with `hg serve -S` or hgwebdir with subrepo/virtual paths that start with 'api'?  (This was the reason I used the LFS URI starting with '.hg/', knowing that isn't committable.)
  
  
  Bleh.
  
  So what I'm hearing is that because of hgwebdir's virtual paths and support for serving subrepos, pretty much the entire URL space is dangerous because it can conflict with the path of a virtual repo or subrepo. I suppose we already have problems with existing URLs, like `/rev/`.
  
  I agree that putting things under `.hg/` in the URL space seems to be a viable workaround.
  
  Changing the URL prefix isn't a big deal as far as the code is concerned. We can make that change at any time since we have no BC guarantees at this juncture.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D2834

To: indygreg, #hg-reviewers
Cc: mharbison72, mercurial-devel


More information about the Mercurial-devel mailing list