After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 782896 - gitlab apache should not decode url
gitlab apache should not decode url
Status: RESOLVED FIXED
Product: sysadmin
Classification: Infrastructure
Component: Other
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GNOME Sysadmins
GNOME Sysadmins
Depends on:
Blocks:
 
 
Reported: 2017-05-21 08:27 UTC by Alberto Fanjul
Modified: 2017-05-25 13:55 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Alberto Fanjul 2017-05-21 08:27:09 UTC
Recommendation from gitlab: https://github.com/gitlabhq/gitlab-recipes/blob/master/web-server/apache/gitlab-apache22.conf#L20

there's an api call that needs to resolve api/v4/projects/user<html tag for slash>project

and seems gitlab.gnome.org is decoding that and looking for api/v4/projects/user∕project
Comment 1 Andrea Veri 2017-05-24 15:47:58 UTC
Done.
Comment 2 Alberto Fanjul 2017-05-24 20:25:49 UTC
Seems it is not working yet.

Test:

$ curl -s https://gitlab.gnome.org/api/v4/projects | jq '.[] | select(.path_with_namespace == "albfan/gnome-todo")'
{
  "id": 9,
  "http_url_to_repo": "http://gitlab.gnome.org/albfan/gnome-todo.git",
  "web_url": "http://gitlab.gnome.org/albfan/gnome-todo",
  "name": "gnome-todo",
  "name_with_namespace": "Alberto / gnome-todo",
  "path": "gnome-todo",
  "path_with_namespace": "albfan/gnome-todo"
}
$ curl -s https://gitlab.gnome.org/api/v4/projects/9 | jq
{
  "id": 9,
  "http_url_to_repo": "http://gitlab.gnome.org/albfan/gnome-todo.git",
  "web_url": "http://gitlab.gnome.org/albfan/gnome-todo",
  "name": "gnome-todo",
  "name_with_namespace": "Alberto / gnome-todo",
  "path": "gnome-todo",
  "path_with_namespace": "albfan/gnome-todo"
}
$ curl -s https://gitlab.gnome.org/api/v4/projects/albfan%2Fgnome-todo | jq
{
  "message": "404 Project Not Found"
}
Comment 3 Andrea Veri 2017-05-25 13:35:09 UTC
Should be really fixed now. Please confirm and close accordingly.
Comment 4 Alberto Fanjul 2017-05-25 13:55:45 UTC
Fixed and tested