{"action":"created","issue":{"url":"https://api.github.com/repos/SparkPost/ruby-sparkpost/issues/11","labels_url":"https://api.github.com/repos/SparkPost/ruby-sparkpost/issues/11/labels{/name}","comments_url":"https://api.github.com/repos/SparkPost/ruby-sparkpost/issues/11/comments","events_url":"https://api.github.com/repos/SparkPost/ruby-sparkpost/issues/11/events","html_url":"https://github.com/SparkPost/ruby-sparkpost/issues/11","id":124307135,"number":11,"title":"Validate email addresses","user":{"login":"rajumsys","id":14877079,"avatar_url":"https://avatars.githubusercontent.com/u/14877079?v=3","gravatar_id":"","url":"https://api.github.com/users/rajumsys","html_url":"https://github.com/rajumsys","followers_url":"https://api.github.com/users/rajumsys/followers","following_url":"https://api.github.com/users/rajumsys/following{/other_user}","gists_url":"https://api.github.com/users/rajumsys/gists{/gist_id}","starred_url":"https://api.github.com/users/rajumsys/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rajumsys/subscriptions","organizations_url":"https://api.github.com/users/rajumsys/orgs","repos_url":"https://api.github.com/users/rajumsys/repos","events_url":"https://api.github.com/users/rajumsys/events{/privacy}","received_events_url":"https://api.github.com/users/rajumsys/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/SparkPost/ruby-sparkpost/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":{"login":"AimeeKnight","id":2165184,"avatar_url":"https://avatars.githubusercontent.com/u/2165184?v=3","gravatar_id":"","url":"https://api.github.com/users/AimeeKnight","html_url":"https://github.com/AimeeKnight","followers_url":"https://api.github.com/users/AimeeKnight/followers","following_url":"https://api.github.com/users/AimeeKnight/following{/other_user}","gists_url":"https://api.github.com/users/AimeeKnight/gists{/gist_id}","starred_url":"https://api.github.com/users/AimeeKnight/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AimeeKnight/subscriptions","organizations_url":"https://api.github.com/users/AimeeKnight/orgs","repos_url":"https://api.github.com/users/AimeeKnight/repos","events_url":"https://api.github.com/users/AimeeKnight/events{/privacy}","received_events_url":"https://api.github.com/users/AimeeKnight/received_events","type":"User","site_admin":false},"milestone":null,"comments":4,"created_at":"2015-12-30T05:49:24Z","updated_at":"2016-01-05T20:52:48Z","closed_at":null,"body":"We need to validate recipient (`to`) and sender (`from`) email addresses in `Transmission` class. "},"comment":{"url":"https://api.github.com/repos/SparkPost/ruby-sparkpost/issues/comments/169130401","html_url":"https://github.com/SparkPost/ruby-sparkpost/issues/11#issuecomment-169130401","issue_url":"https://api.github.com/repos/SparkPost/ruby-sparkpost/issues/11","id":169130401,"user":{"login":"AimeeKnight","id":2165184,"avatar_url":"https://avatars.githubusercontent.com/u/2165184?v=3","gravatar_id":"","url":"https://api.github.com/users/AimeeKnight","html_url":"https://github.com/AimeeKnight","followers_url":"https://api.github.com/users/AimeeKnight/followers","following_url":"https://api.github.com/users/AimeeKnight/following{/other_user}","gists_url":"https://api.github.com/users/AimeeKnight/gists{/gist_id}","starred_url":"https://api.github.com/users/AimeeKnight/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AimeeKnight/subscriptions","organizations_url":"https://api.github.com/users/AimeeKnight/orgs","repos_url":"https://api.github.com/users/AimeeKnight/repos","events_url":"https://api.github.com/users/AimeeKnight/events{/privacy}","received_events_url":"https://api.github.com/users/AimeeKnight/received_events","type":"User","site_admin":false},"created_at":"2016-01-05T20:52:48Z","updated_at":"2016-01-05T20:52:48Z","body":"@rajumsys after talking through things with a few others who first developed the node library, I think we should just allow the api to surface the errors as you showed above. I definitely don't see an issue with that error message, it also prevents us from having to maintain unnecessary code, and keeps us consistent with the other libraries. Validating a recipient list and not a single recipient seems unintuitive from a maintainers standpoint, and couples our library pretty tightly with the API. Depending on the error surfaced, this could also cause confusion from a consumer standpoint if we surface errors that don't match the api exactly since it's hard to predict what kind of errors a user might see. Let me know what you think."}}