{"action":"created","issue":{"url":"https://api.github.com/repos/json-schema-org/JSON-Schema-Test-Suite/issues/217","repository_url":"https://api.github.com/repos/json-schema-org/JSON-Schema-Test-Suite","labels_url":"https://api.github.com/repos/json-schema-org/JSON-Schema-Test-Suite/issues/217/labels{/name}","comments_url":"https://api.github.com/repos/json-schema-org/JSON-Schema-Test-Suite/issues/217/comments","events_url":"https://api.github.com/repos/json-schema-org/JSON-Schema-Test-Suite/issues/217/events","html_url":"https://github.com/json-schema-org/JSON-Schema-Test-Suite/issues/217","id":308370279,"number":217,"title":"Draft 4 - recursive references test requires internal referencing","user":{"login":"yuloh","id":9440455,"avatar_url":"https://avatars0.githubusercontent.com/u/9440455?v=4","gravatar_id":"","url":"https://api.github.com/users/yuloh","html_url":"https://github.com/yuloh","followers_url":"https://api.github.com/users/yuloh/followers","following_url":"https://api.github.com/users/yuloh/following{/other_user}","gists_url":"https://api.github.com/users/yuloh/gists{/gist_id}","starred_url":"https://api.github.com/users/yuloh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yuloh/subscriptions","organizations_url":"https://api.github.com/users/yuloh/orgs","repos_url":"https://api.github.com/users/yuloh/repos","events_url":"https://api.github.com/users/yuloh/events{/privacy}","received_events_url":"https://api.github.com/users/yuloh/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2018-03-25T17:53:43Z","updated_at":"2018-03-26T16:23:53Z","closed_at":"2018-03-26T16:23:53Z","author_association":"CONTRIBUTOR","body":"Hello,\r\n\r\nI updated the test suite which pulled in [this test](https://github.com/json-schema-org/JSON-Schema-Test-Suite/blob/71843cbbd4be3194ee83253b402656550c8c0522/tests/draft4/ref.json#L213).\r\n\r\nThe schema's id is `http://localhost:1234/tree` and it contains `{\"$ref\": \"node\"}`.  When my validator encounters the reference it resolves to `http://localhost:1234/node`.  At this point it attempts to load `http://localhost:1234/node` and fails because that schema cannot be loaded via HTTP.\r\n\r\nI see that `http://localhost:1234/node` is defined below within the same schema but I don't see which part of the Draft-04 spec says that you should attempt to load the schema from within the same document first.\r\n\r\nIt looks like Draft-05 covers that in [section 8.2.1](https://tools.ietf.org/html/draft-wright-json-schema-00#section-8.2.1) but that section does not exist in Draft-04.  The closest thing I see is [section 7.2.3](https://tools.ietf.org/html/draft-zyp-json-schema-04#section-7.2.3).  It seems like this might be considered inline dereferencing but that isn't required by the spec.\r\n\r\nAm I interpreting the spec incorrectly or should this behavior not be required for Draft-04?"},"comment":{"url":"https://api.github.com/repos/json-schema-org/JSON-Schema-Test-Suite/issues/comments/376225937","html_url":"https://github.com/json-schema-org/JSON-Schema-Test-Suite/issues/217#issuecomment-376225937","issue_url":"https://api.github.com/repos/json-schema-org/JSON-Schema-Test-Suite/issues/217","id":376225937,"user":{"login":"yuloh","id":9440455,"avatar_url":"https://avatars0.githubusercontent.com/u/9440455?v=4","gravatar_id":"","url":"https://api.github.com/users/yuloh","html_url":"https://github.com/yuloh","followers_url":"https://api.github.com/users/yuloh/followers","following_url":"https://api.github.com/users/yuloh/following{/other_user}","gists_url":"https://api.github.com/users/yuloh/gists{/gist_id}","starred_url":"https://api.github.com/users/yuloh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yuloh/subscriptions","organizations_url":"https://api.github.com/users/yuloh/orgs","repos_url":"https://api.github.com/users/yuloh/repos","events_url":"https://api.github.com/users/yuloh/events{/privacy}","received_events_url":"https://api.github.com/users/yuloh/received_events","type":"User","site_admin":false},"created_at":"2018-03-26T16:23:53Z","updated_at":"2018-03-26T16:23:53Z","author_association":"CONTRIBUTOR","body":"Oh, I should have mentioned that the test in question was merged in commit 0178c74c894f6b1a2f7c9ac6f9b60da44e786b6f.  I had the test suite locked to the last tag in my dependency manager so it wasn't pulled in until I switched to master.  Since it's been in here for a year now I am fine with leaving it and skipping the test.  I know 2 PHP validators have been written in PHP since it was merged that now pass that test, so it seems likely that quite a few implementations have implemented that behavior by now.\r\n\r\nThanks for the feedback 👍"}}
{"actor":{"display_login":"yuloh"}}