{"action":"created","issue":{"url":"https://api.github.com/repos/openwrt/packages/issues/3223","repository_url":"https://api.github.com/repos/openwrt/packages","labels_url":"https://api.github.com/repos/openwrt/packages/issues/3223/labels{/name}","comments_url":"https://api.github.com/repos/openwrt/packages/issues/3223/comments","events_url":"https://api.github.com/repos/openwrt/packages/issues/3223/events","html_url":"https://github.com/openwrt/packages/pull/3223","id":179024678,"number":3223,"title":"ddns-scripts: bump to version 2.7.5","user":{"login":"chris5560","id":8249658,"avatar_url":"https://avatars.githubusercontent.com/u/8249658?v=3","gravatar_id":"","url":"https://api.github.com/users/chris5560","html_url":"https://github.com/chris5560","followers_url":"https://api.github.com/users/chris5560/followers","following_url":"https://api.github.com/users/chris5560/following{/other_user}","gists_url":"https://api.github.com/users/chris5560/gists{/gist_id}","starred_url":"https://api.github.com/users/chris5560/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chris5560/subscriptions","organizations_url":"https://api.github.com/users/chris5560/orgs","repos_url":"https://api.github.com/users/chris5560/repos","events_url":"https://api.github.com/users/chris5560/events{/privacy}","received_events_url":"https://api.github.com/users/chris5560/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2016-09-24T10:59:33Z","updated_at":"2016-09-27T08:32:06Z","closed_at":"2016-09-27T08:18:18Z","pull_request":{"url":"https://api.github.com/repos/openwrt/packages/pulls/3223","html_url":"https://github.com/openwrt/packages/pull/3223","diff_url":"https://github.com/openwrt/packages/pull/3223.diff","patch_url":"https://github.com/openwrt/packages/pull/3223.patch"},"body":"Maintainer: me\r\nCompile tested: DD trunk \r\nRun tested: DD trunk x86 and WNDR3800\r\n\r\nDescription:\r\n- commands to apply changes introduced during release changes, moved from Makefile postinst to /etc/uci-defaults/ddns\r\n- do not break execution if using nslookup and force_ipversion=1\r\n- add support of knsupdate (update_nsupdate.sh\r\n- url update easydns.com\r\n- add some service answers\r\n- alphabetic reorder services files for easier reading\r\n- rename services/provider (Makefile postinst handle /etc/config/ddns)\r\n\r\n1. freedns.afraid.org -> afraid.org\r\n2. free.editdns.net -> editdns.net\r\n3. domains.google.com -> google.com\r\n4. spdns.de -> spdyn.de\r\n5. strato.de -> strato.com\r\n\r\n- new provider (looking in deep into https://sourceforge.net/projects/inadyn-mt project)\r\n\r\n1. dyn.com (= dyndns.org)\r\n2. ddnss.de\r\n3. dhis.org\r\n4. dnspark.com (IPv4 only)\r\n5. dynsip.org (IPv4 only)\r\n6. dynv6.com\r\n7. joker.com (IPv4 only)\r\n8. loopia.com (= loopia.se)\r\n9. sitelutions.com (IPv4 only)\r\n10. system-ns.com (IPv4 only)\r\n\r\n- new provider (looking in deep into /etc.defaults/ddns_provider.conf file for Synology DiskStation published at https://gist.github.com/ntrepid8/6653274)\r\n\r\n1. able.or.kr (IPv4 only)\r\n2. ddo.jp (IPv4 only)\r\n3. dnsmadeeasy.com (IPv4 only)\r\n4. oray.com (IPv4 only)\r\n\r\n- new provider (looking in deep into https://github.com/ipfire/ddns project)\r\n\r\n1. all-inkl.com\r\n2. desec.io\r\n3. domopoli.de (IPv4 only)\r\n4. opendns.com (IPv4 only)\r\n5. udmedia.de\r\n6. xlhost.de (IPv4 only)\r\n\r\n- new provider (looking in deep into https://github.com/yaddns/yaddns project )\r\n\r\n1. dyndns.it (IPv4 only)\r\n\r\nSigned-off-by: Christian Schoenebeck <<christian.schoenebeck@gmail.com>>"},"comment":{"url":"https://api.github.com/repos/openwrt/packages/issues/comments/249800909","html_url":"https://github.com/openwrt/packages/pull/3223#issuecomment-249800909","issue_url":"https://api.github.com/repos/openwrt/packages/issues/3223","id":249800909,"user":{"login":"hnyman","id":7926856,"avatar_url":"https://avatars.githubusercontent.com/u/7926856?v=3","gravatar_id":"","url":"https://api.github.com/users/hnyman","html_url":"https://github.com/hnyman","followers_url":"https://api.github.com/users/hnyman/followers","following_url":"https://api.github.com/users/hnyman/following{/other_user}","gists_url":"https://api.github.com/users/hnyman/gists{/gist_id}","starred_url":"https://api.github.com/users/hnyman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hnyman/subscriptions","organizations_url":"https://api.github.com/users/hnyman/orgs","repos_url":"https://api.github.com/users/hnyman/repos","events_url":"https://api.github.com/users/hnyman/events{/privacy}","received_events_url":"https://api.github.com/users/hnyman/received_events","type":"User","site_admin":false},"created_at":"2016-09-27T08:32:06Z","updated_at":"2016-09-27T08:32:06Z","body":"I merged it.\r\n\r\nOne thing that you might need to enhance in the next version: \r\nThe current code may cause /etc/config/ddns to get unnecessarily edited by 'sed' after each subsequent firmware flash due to uci-defaults being re-run after each flash. I am not quite sure, but it looks like that may happen. uci is smart and does not edit unnecessarily, but sed is likely a brute-force tool.\r\n\r\nYou might first evaluate the need for changes e.g. by grep for \"freedns.afraid.org\" and then use 'sed' only if needed. \r\n\r\nOr you might take the 'adblock' approach, where the config file is rather similar as it defines the blocklists. There the config has a separate version number that is evaluated at the service startup and depending on the seriousness of the config change, the user may be either \r\n* prompted in the log to update his config and the service startup is breaked, or \r\n* notified about \"partially outdated config\" and is encouraged to update his config.\r\n"}}
{"actor":{"display_login":"hnyman"}}