{"action":"created","issue":{"url":"https://api.github.com/repos/projectatomic/atomic/issues/125","labels_url":"https://api.github.com/repos/projectatomic/atomic/issues/125/labels{/name}","comments_url":"https://api.github.com/repos/projectatomic/atomic/issues/125/comments","events_url":"https://api.github.com/repos/projectatomic/atomic/issues/125/events","html_url":"https://github.com/projectatomic/atomic/pull/125","id":102790047,"number":125,"title":"Atomic.util.image_by_name only matches some possible matches","user":{"login":"ibotty","id":296430,"avatar_url":"https://avatars.githubusercontent.com/u/296430?v=3","gravatar_id":"","url":"https://api.github.com/users/ibotty","html_url":"https://github.com/ibotty","followers_url":"https://api.github.com/users/ibotty/followers","following_url":"https://api.github.com/users/ibotty/following{/other_user}","gists_url":"https://api.github.com/users/ibotty/gists{/gist_id}","starred_url":"https://api.github.com/users/ibotty/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ibotty/subscriptions","organizations_url":"https://api.github.com/users/ibotty/orgs","repos_url":"https://api.github.com/users/ibotty/repos","events_url":"https://api.github.com/users/ibotty/events{/privacy}","received_events_url":"https://api.github.com/users/ibotty/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2015-08-24T13:11:40Z","updated_at":"2015-10-02T21:52:01Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/projectatomic/atomic/pulls/125","html_url":"https://github.com/projectatomic/atomic/pull/125","diff_url":"https://github.com/projectatomic/atomic/pull/125.diff","patch_url":"https://github.com/projectatomic/atomic/pull/125.patch"},"body":"e.g. the following image (in docker)\r\n\r\n    docker.io/user/image:tag\r\n\r\nis not matched by the following queries, because the first `/` might not separate the registry from the user-image but the user from the image. \r\n\r\n * user/image\r\n * image\r\n\r\nThe fix for that case is straightforward, but the catch is, that non-prefixed containers (e.g. docker.io/fedora:latest) show an ambiguity: It could be user `docker.io` or registry `docker.io`.\r\n\r\nNon-user supplied queries to `_decompose` might also be incorrectly matched, because (at least on one of my machines) there were images without registry: `user/image:latest` in docker's images output.\r\n\r\nThe patch fixes it by not decomposing the query (and docker's images), but directly matching four times."},"comment":{"url":"https://api.github.com/repos/projectatomic/atomic/issues/comments/145164380","html_url":"https://github.com/projectatomic/atomic/pull/125#issuecomment-145164380","issue_url":"https://api.github.com/repos/projectatomic/atomic/issues/125","id":145164380,"user":{"login":"jlebon","id":11934099,"avatar_url":"https://avatars.githubusercontent.com/u/11934099?v=3","gravatar_id":"","url":"https://api.github.com/users/jlebon","html_url":"https://github.com/jlebon","followers_url":"https://api.github.com/users/jlebon/followers","following_url":"https://api.github.com/users/jlebon/following{/other_user}","gists_url":"https://api.github.com/users/jlebon/gists{/gist_id}","starred_url":"https://api.github.com/users/jlebon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jlebon/subscriptions","organizations_url":"https://api.github.com/users/jlebon/orgs","repos_url":"https://api.github.com/users/jlebon/repos","events_url":"https://api.github.com/users/jlebon/events{/privacy}","received_events_url":"https://api.github.com/users/jlebon/received_events","type":"User","site_admin":false},"created_at":"2015-10-02T21:52:01Z","updated_at":"2015-10-02T21:52:01Z","body":"bot, retest this please"}}