{"action":"created","issue":{"url":"https://api.github.com/repos/spark/core-communication-lib/issues/26","labels_url":"https://api.github.com/repos/spark/core-communication-lib/issues/26/labels{/name}","comments_url":"https://api.github.com/repos/spark/core-communication-lib/issues/26/comments","events_url":"https://api.github.com/repos/spark/core-communication-lib/issues/26/events","html_url":"https://github.com/spark/core-communication-lib/issues/26","id":41876958,"number":26,"title":"Spark.variable accesses may overlap","user":{"login":"towynlin","id":3389,"avatar_url":"https://avatars.githubusercontent.com/u/3389?v=3","gravatar_id":"","url":"https://api.github.com/users/towynlin","html_url":"https://github.com/towynlin","followers_url":"https://api.github.com/users/towynlin/followers","following_url":"https://api.github.com/users/towynlin/following{/other_user}","gists_url":"https://api.github.com/users/towynlin/gists{/gist_id}","starred_url":"https://api.github.com/users/towynlin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/towynlin/subscriptions","organizations_url":"https://api.github.com/users/towynlin/orgs","repos_url":"https://api.github.com/users/towynlin/repos","events_url":"https://api.github.com/users/towynlin/events{/privacy}","received_events_url":"https://api.github.com/users/towynlin/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/spark/core-communication-lib/labels/bug","name":"bug","color":"fc2929"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2014-09-03T22:49:30Z","updated_at":"2015-03-26T15:32:11Z","closed_at":null,"body":"We need a test to prove this, but there's good evidence that it happens:\r\n\r\n- https://community.spark.io/t/two-variables-return-the-same-value-periodically/5220\r\n- http://community.spark.io/t/spark-variable-frequent-call/6452\r\n"},"comment":{"url":"https://api.github.com/repos/spark/core-communication-lib/issues/comments/86572251","html_url":"https://github.com/spark/core-communication-lib/issues/26#issuecomment-86572251","issue_url":"https://api.github.com/repos/spark/core-communication-lib/issues/26","id":86572251,"user":{"login":"andyw-lala","id":3930944,"avatar_url":"https://avatars.githubusercontent.com/u/3930944?v=3","gravatar_id":"","url":"https://api.github.com/users/andyw-lala","html_url":"https://github.com/andyw-lala","followers_url":"https://api.github.com/users/andyw-lala/followers","following_url":"https://api.github.com/users/andyw-lala/following{/other_user}","gists_url":"https://api.github.com/users/andyw-lala/gists{/gist_id}","starred_url":"https://api.github.com/users/andyw-lala/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andyw-lala/subscriptions","organizations_url":"https://api.github.com/users/andyw-lala/orgs","repos_url":"https://api.github.com/users/andyw-lala/repos","events_url":"https://api.github.com/users/andyw-lala/events{/privacy}","received_events_url":"https://api.github.com/users/andyw-lala/received_events","type":"User","site_admin":false},"created_at":"2015-03-26T15:32:11Z","updated_at":"2015-03-26T15:32:11Z","body":"Well, the cloud obviously isn't checking/enforcing them either, so I think\nfixing the firmware is required, but not sufficient.\n\nCould be a tricky fix to roll out, if that is the case, perhaps with the\ncloud using a heuristic to figure out if it should enforce (new FW) or not\n(old FW).\n\nOn Thu, Mar 26, 2015 at 10:28 AM, David Middlecamp <notifications@github.com\n> wrote:\n\n> The protocol provides for this by tagging each coap message with both a\n> message id and token. These are supposed to be used when sending responses\n> back to the cloud. The communications lib in the firmware doesn't tag\n> variable responses with the corresponding request id / token, so it's\n> impossible for the cloud to distinguish which response goes with which\n> request. So it's a pretty clear \"the firmware isn't following the protocol\"\n> scenario I think.\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/spark/core-communication-lib/issues/26#issuecomment-86569265>\n> .\n>\n\n\n\n-- \nAndy\n"}}