{"action":"created","issue":{"url":"https://api.github.com/repos/FasterXML/jackson-databind/issues/1261","repository_url":"https://api.github.com/repos/FasterXML/jackson-databind","labels_url":"https://api.github.com/repos/FasterXML/jackson-databind/issues/1261/labels{/name}","comments_url":"https://api.github.com/repos/FasterXML/jackson-databind/issues/1261/comments","events_url":"https://api.github.com/repos/FasterXML/jackson-databind/issues/1261/events","html_url":"https://github.com/FasterXML/jackson-databind/issues/1261","id":158840328,"number":1261,"title":"JsonIdentityInfo broken deserialization involving forward references and/or cycles","user":{"login":"arifogel","id":7737874,"avatar_url":"https://avatars.githubusercontent.com/u/7737874?v=3","gravatar_id":"","url":"https://api.github.com/users/arifogel","html_url":"https://github.com/arifogel","followers_url":"https://api.github.com/users/arifogel/followers","following_url":"https://api.github.com/users/arifogel/following{/other_user}","gists_url":"https://api.github.com/users/arifogel/gists{/gist_id}","starred_url":"https://api.github.com/users/arifogel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/arifogel/subscriptions","organizations_url":"https://api.github.com/users/arifogel/orgs","repos_url":"https://api.github.com/users/arifogel/repos","events_url":"https://api.github.com/users/arifogel/events{/privacy}","received_events_url":"https://api.github.com/users/arifogel/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2016-06-07T05:16:09Z","updated_at":"2016-06-08T02:38:03Z","closed_at":null,"body":"As a follow-up to bug #1255, the patch I provided exposes related deserialization problems.\r\nI have attached a small project ('jackson-test.zip') to demonstrate these issues. When run with both patches from #1255, the output is provided in the attached 'both.txt'. When run with just the first patch from #1255, the output is provided in the attached 'first.txt'.\r\nImportant points:\r\n1. When the object expressed as an id is contained within a collection or map (List in this example), deserialization works correctly. When it is a field of an object, deserialization is broken.\r\n2. This particular example doesn't have forward references, but it does have cycles. Nevertheless, I have seen situations where non-cyclical forward-references also do not deserialize properly, with the same caveat as in 1.\r\n[jackson-test.zip](https://github.com/FasterXML/jackson-databind/files/301884/jackson-test.zip)\r\n[both.txt](https://github.com/FasterXML/jackson-databind/files/301885/both.txt)\r\n[first.txt](https://github.com/FasterXML/jackson-databind/files/301886/first.txt)\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/FasterXML/jackson-databind/issues/comments/224471982","html_url":"https://github.com/FasterXML/jackson-databind/issues/1261#issuecomment-224471982","issue_url":"https://api.github.com/repos/FasterXML/jackson-databind/issues/1261","id":224471982,"user":{"login":"cowtowncoder","id":55065,"avatar_url":"https://avatars.githubusercontent.com/u/55065?v=3","gravatar_id":"","url":"https://api.github.com/users/cowtowncoder","html_url":"https://github.com/cowtowncoder","followers_url":"https://api.github.com/users/cowtowncoder/followers","following_url":"https://api.github.com/users/cowtowncoder/following{/other_user}","gists_url":"https://api.github.com/users/cowtowncoder/gists{/gist_id}","starred_url":"https://api.github.com/users/cowtowncoder/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cowtowncoder/subscriptions","organizations_url":"https://api.github.com/users/cowtowncoder/orgs","repos_url":"https://api.github.com/users/cowtowncoder/repos","events_url":"https://api.github.com/users/cowtowncoder/events{/privacy}","received_events_url":"https://api.github.com/users/cowtowncoder/received_events","type":"User","site_admin":false},"created_at":"2016-06-08T02:38:03Z","updated_at":"2016-06-08T02:38:03Z","body":"Lack of catching for beans is particularly puzzling since that is the main use case.\r\nSo going back to the original pr, #388 (and issue #351) (they were missing from release notes for reason, added), I think that bean property handling should be included via `ObjectIdReferenceProperty`, and question would be why this isn't triggered in test cases. I will try to see what tests uncover.\r\n"}}
{"actor":{"display_login":"cowtowncoder"}}