Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Bad links with engine mounted as root

Đang mở
#1,303 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
45/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
rails, ruby
Lĩnh vực
api, backend

Hướng nghiên cứu

Bắt đầu trong lib/jsonapi/link_builder.rb tại resource_url, sau đó kiểm tra cách engine_mount_point được tính toán cho một engine được mount tại '/'. Tái hiện trường hợp root-mounted được báo cáo và xác minh rằng self link được tạo ra có một dấu gạch chéo ở đầu thay vì '//v1/users/...'; hành vi xây dựng link hiện có vẫn phải chính xác đối với các mount không phải root.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

This issue is a (choose one):

  • Problem/bug report.
  • Feature request.
  • Request for support. Note: Please try to avoid submitting issues for support requests. Use Gitter instead.

Checklist before submitting:

  • I've searched for an existing issue.
  • I've asked my question on Gitter and have not received a satisfactory answer.
  • I've included a complete bug report template. This step helps us and allows us to see the bug without trying to reproduce the problem from your description. It helps you because you will frequently detect if it's a problem specific to your project.
  • The feature I'm asking for is compliant with the JSON:API spec.

Description

As described on lib/jsonapi/link_builder.rb#L139,

def resource_url(source)
    "#{ base_url }#{ engine_mount_point }#{ resource_path(source) }"
end

But in this case:

constraints subdomain: 'api' do
    mount MyEngine::Engine, at: '/' if defined?(MyEngine::Engine)
  end

This creates a double / due to root value of engine_mount_point.

I guess a little update of:

@engine_mount_point = @engine ? @engine.routes.find_script_name({}) : ""

should be enough.

Current result:

{
    "data": {
        "id": "ed526d56-cc7e-41f4-8e99-fccf73fb7500",
        "type": "users",
        "links": {
            "self": "//v1/users/ed526d56-cc7e-41f4-8e99-fccf73fb7500"
        },
        "attributes": {
            "first_name": null,
            "last_name": null,
            "email": "some@thing.io"
        }
    }
}

//v1/users/ed526d56-cc7e-41f4-8e99-fccf73fb7500 should be /v1/users/ed526d56-cc7e-41f4-8e99-fccf73fb7500

Ngôn ngữ chính
Ruby
Star
2.3k
Fork
546
Không có pull request nào được merge trong 30 ngày

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của JSONAPI-Resources/jsonapi-resources

Tất cả issue của JSONAPI-Resources/jsonapi-resources

Issue tương tự

Thêm issue về Ruby

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.