Page MenuHomePhabricator

urls link to locahost, and should probably link to https://techcontribs.toolforge.org/
Closed, ResolvedPublic

Description

https://techcontribs.toolforge.org/api/mail/foobar@baz.com

{
  "error": false,
  "accounts": [],
  "urls": {
    "gerrit": "https://localhost:8000/api/mail/foobar@baz.com/gerrit",
    "phabricator": "https://localhost:8000/api/mail/foobar@baz.com/phabricator"
  }
}

https://techcontribs.toolforge.org//api/mail/foobar@baz.com/gerrit

Event Timeline

Chlod changed the task status from Open to In Progress.Mar 14 2026, 5:19 PM
Chlod triaged this task as Medium priority.
Chlod moved this task from Backlog to To Deploy on the Tool-techcontribs board.
Chlod subscribed.

These URLs actually should be removed from the response; the only ones that have them are the /api/(uid|cn|mail) endpoints, and I initially intended all normal endpoints to have that but it seemed a bit excessive (API docs felt like a better place to find these than throwing blind requests to the API). That said, I have no clue if people actively rely on this, so I'm willing to give some very minor support for it while there isn't a "real" API docs page for the tool yet.

In any case, this seems to have been a problem that would only show up when using Tech Contribs via a reverse proxy (i.e., Toolforge). I've added some basic reverse proxy support which should make these URLs point to the right place.

Chlod claimed this task.
Chlod moved this task from To Deploy to Done on the Tool-techcontribs board.