{"comment":{"user":{"id":18057,"login":"benadida","url":"https://api.github.com/users/benadida","gravatar_id":"e7591fc2ef5b6733035b570ae7942b47","avatar_url":"https://secure.gravatar.com/avatar/e7591fc2ef5b6733035b570ae7942b47?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"created_at":"2011-12-22T23:20:15Z","path":"lib/verifier/certassertion.js","updated_at":"2011-12-22T23:20:15Z","id":817076,"html_url":"https://github.com/mozilla/browserid/commit/2a342400d2#commitcomment-817076","position":19,"body":"this code is doing the right thing, but the variable naming and the comment appear to be off. This function is only called once with the root issuer of the chain, not each time with each certificate in the chain. So ultimateIssuer should probably be rootIssuer, and you should expect it to be set precisely once (this function will be called exactly once.)","url":"https://api.github.com/repos/mozilla/browserid/comments/817076","line":135,"commit_id":"2a342400d29a59dd6d20502bafce72fcc94d5c60"}}