Skip to content
Snippets Groups Projects

Fix 404 error while displaying json files.

Merged gitlab-qa-bot requested to merge github/fork/hiroponz/fix-404-json-file into master

Created by: hiroponz

It uses params[:id] instead of request.fullpath. It should fix #3132 (closed).

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Created by: coveralls

    Coverage increased (+12.38%) when pulling a1fe375e on hiroponz:fix-404-json-file into 02f70851 on gitlabhq:master.

    View Details

    By Administrator on 2013-03-21T16:18:42 (imported from GitLab project)

    By Administrator on 2013-03-21T16:18:42 (imported from GitLab)

  • Created by: zzet

    :thumbsup:

    #<ActionDispatch::Request:0x00000004aae160
    @env={"REMOTE_ADDR"=>"127.0.0.1", "REQUEST_METHOD"=>"GET", "REQUEST_PATH"=>"/project_path_with_namespace/tree/master/full/path/to/file.json", "PATH_INFO"=>"/project_path_with_namespace">
    @request_method="GET",
    @filtered_parameters={"action"=>"show", "controller"=>"tree", "project_id"=>"project_path_with_namespace", "id"=>"master/full/path/to/file.json"},
    @method="GET",
    @fullpath="/project_path_with_namespace/tree/master/full/path/to/file.json">

    By Administrator on 2013-03-24T08:42:07 (imported from GitLab project)

    By Administrator on 2013-03-24T08:42:07 (imported from GitLab)

  • Created by: dzaporozhets

    thank you

    By Administrator on 2013-03-24T10:32:39 (imported from GitLab project)

    By Administrator on 2013-03-24T10:32:39 (imported from GitLab)

Please register or sign in to reply
Loading