{"action":"opened","issue":{"url":"https://api.github.com/repos/jetstack/cert-manager/issues/857","repository_url":"https://api.github.com/repos/jetstack/cert-manager","labels_url":"https://api.github.com/repos/jetstack/cert-manager/issues/857/labels{/name}","comments_url":"https://api.github.com/repos/jetstack/cert-manager/issues/857/comments","events_url":"https://api.github.com/repos/jetstack/cert-manager/issues/857/events","html_url":"https://github.com/jetstack/cert-manager/issues/857","id":354501989,"node_id":"MDU6SXNzdWUzNTQ1MDE5ODk=","number":857,"title":"panic from acmedns.go failed constructor","user":{"login":"jjo","id":88727,"node_id":"MDQ6VXNlcjg4NzI3","avatar_url":"https://avatars2.githubusercontent.com/u/88727?v=4","gravatar_id":"","url":"https://api.github.com/users/jjo","html_url":"https://github.com/jjo","followers_url":"https://api.github.com/users/jjo/followers","following_url":"https://api.github.com/users/jjo/following{/other_user}","gists_url":"https://api.github.com/users/jjo/gists{/gist_id}","starred_url":"https://api.github.com/users/jjo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jjo/subscriptions","organizations_url":"https://api.github.com/users/jjo/orgs","repos_url":"https://api.github.com/users/jjo/repos","events_url":"https://api.github.com/users/jjo/events{/privacy}","received_events_url":"https://api.github.com/users/jjo/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-08-27T22:30:53Z","updated_at":"2018-08-27T22:30:53Z","closed_at":null,"author_association":"NONE","body":"> Bugs should be filed for issues encountered whilst operating cert-manager.\r\n> You should first attempt to resolve your issues through the community support\r\n> channels, e.g. Slack, in order to rule out individual configuration errors.\r\n> Please provide as much detail as possible.\r\n\r\n**Describe the bug**:\r\ncert-manager panics if `acmedns.go:NewDNSProviderHostBytes()` fails to construct `DNSProvider`,\r\nhappens e.g. if `acmedns.json` from kube secret is invalid.\r\n\r\n**Expected behaviour**:\r\ncert-manager should warn and skip `acmedns` initialization, instead of panic-ing.\r\n\r\n**Steps to reproduce the bug**:\r\n\r\nFollow https://github.com/jetstack/cert-manager/blob/master/docs/reference/issuers/acme/dns01.rst#acme-dns,\r\nprovide an invalid `acmedns.json`.\r\n\r\n**Anything else we need to know?**:\r\nI'll be shortly providing the fixing PR.\r\n\r\n**Environment details:**:\r\n- Kubernetes v1.11.2\r\n- Cloud-provider/provisioner: metal on ubuntu xenial nodes\r\n- cert-manager version: `master-5096` (canary as of 2018-08-27)\r\n- Install method: helm + manual mods for image version\r\n\r\n/kind bug\r\n"}}
{"actor":{"display_login":"jjo"}}