{"action":"created","issue":{"url":"https://api.github.com/repos/node-red/node-red-node-swagger/issues/18","labels_url":"https://api.github.com/repos/node-red/node-red-node-swagger/issues/18/labels{/name}","comments_url":"https://api.github.com/repos/node-red/node-red-node-swagger/issues/18/comments","events_url":"https://api.github.com/repos/node-red/node-red-node-swagger/issues/18/events","html_url":"https://github.com/node-red/node-red-node-swagger/issues/18","id":98473579,"number":18,"title":"Flags for unconfigured swagger doc in http nodes","user":{"login":"lostinthestory","id":6432098,"avatar_url":"https://avatars.githubusercontent.com/u/6432098?v=3","gravatar_id":"","url":"https://api.github.com/users/lostinthestory","html_url":"https://github.com/lostinthestory","followers_url":"https://api.github.com/users/lostinthestory/followers","following_url":"https://api.github.com/users/lostinthestory/following{/other_user}","gists_url":"https://api.github.com/users/lostinthestory/gists{/gist_id}","starred_url":"https://api.github.com/users/lostinthestory/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lostinthestory/subscriptions","organizations_url":"https://api.github.com/users/lostinthestory/orgs","repos_url":"https://api.github.com/users/lostinthestory/repos","events_url":"https://api.github.com/users/lostinthestory/events{/privacy}","received_events_url":"https://api.github.com/users/lostinthestory/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-07-31T21:24:55Z","updated_at":"2015-08-07T01:09:30Z","closed_at":null,"body":"The following scenario has been brought up.\r\n\r\n    1) add HTTP In (say /path2) to canvas but don't Add/Edit the Doc settings (aka don't define any response codes)\r\n    2) wire in a function node and add an HTTP Out node\r\n    3) in swagger ui, you can test the API, but there is no definition of the expected response codes or output\r\n\r\nIf you have the swagger plugin enabled, and have http endpoints in your flows, but have not defined any additional documentation via the config, we use a placeholder in the swagger doc generation.\r\n\r\nShould we present the user with a warning flag on the corresponding nodes, informing the user that they do not have any additional swagger doc attached?\r\n\r\nThis can be done in the swagger plugin itself, along with the check for http in nodes, that way we aren't changing the logic in the core nodes themselves.\r\n\r\nThoughts, @knolleary ?"},"comment":{"url":"https://api.github.com/repos/node-red/node-red-node-swagger/issues/comments/128553706","html_url":"https://github.com/node-red/node-red-node-swagger/issues/18#issuecomment-128553706","issue_url":"https://api.github.com/repos/node-red/node-red-node-swagger/issues/18","id":128553706,"user":{"login":"drwoods","id":6426976,"avatar_url":"https://avatars.githubusercontent.com/u/6426976?v=3","gravatar_id":"","url":"https://api.github.com/users/drwoods","html_url":"https://github.com/drwoods","followers_url":"https://api.github.com/users/drwoods/followers","following_url":"https://api.github.com/users/drwoods/following{/other_user}","gists_url":"https://api.github.com/users/drwoods/gists{/gist_id}","starred_url":"https://api.github.com/users/drwoods/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/drwoods/subscriptions","organizations_url":"https://api.github.com/users/drwoods/orgs","repos_url":"https://api.github.com/users/drwoods/repos","events_url":"https://api.github.com/users/drwoods/events{/privacy}","received_events_url":"https://api.github.com/users/drwoods/received_events","type":"User","site_admin":false},"created_at":"2015-08-07T01:09:30Z","updated_at":"2015-08-07T01:09:30Z","body":"+1"}}