Skip to content

Commit 81a51ca

Browse files
authored
Merge pull request #1282 from tobias-grasse/patch-2
Consider config.relative_url_root in ActsAsResourceController.base_url
2 parents 9e18e02 + 37fa80d commit 81a51ca

File tree

2 files changed

+8
-1
lines changed

2 files changed

+8
-1
lines changed

lib/jsonapi/acts_as_resource_controller.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,7 @@ def resource_serializer_klass
152152
end
153153

154154
def base_url
155-
@base_url ||= request.protocol + request.host_with_port
155+
@base_url ||= "#{request.protocol}#{request.host_with_port}#{Rails.application.config.relative_url_root}"
156156
end
157157

158158
def resource_klass_name

test/controllers/controller_test.rb

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,13 @@ def setup
1010
JSONAPI.configuration.always_include_to_one_linkage_data = false
1111
end
1212

13+
def test_links_include_relative_root
14+
Rails.application.config.relative_url_root = '/subdir'
15+
assert_cacheable_get :index
16+
assert json_response['data'][0]['links']['self'].include?('/subdir')
17+
Rails.application.config.relative_url_root = nil
18+
end
19+
1320
def test_index
1421
assert_cacheable_get :index
1522
assert_response :success

0 commit comments

Comments
 (0)