{"action":"opened","issue":{"url":"https://api.github.com/repos/drewr/postal/issues/75","repository_url":"https://api.github.com/repos/drewr/postal","labels_url":"https://api.github.com/repos/drewr/postal/issues/75/labels{/name}","comments_url":"https://api.github.com/repos/drewr/postal/issues/75/comments","events_url":"https://api.github.com/repos/drewr/postal/issues/75/events","html_url":"https://github.com/drewr/postal/issues/75","id":159738552,"number":75,"title":"NullPointerException when no file name in attachment","user":{"login":"arichiardi","id":1724788,"avatar_url":"https://avatars.githubusercontent.com/u/1724788?v=3","gravatar_id":"","url":"https://api.github.com/users/arichiardi","html_url":"https://github.com/arichiardi","followers_url":"https://api.github.com/users/arichiardi/followers","following_url":"https://api.github.com/users/arichiardi/following{/other_user}","gists_url":"https://api.github.com/users/arichiardi/gists{/gist_id}","starred_url":"https://api.github.com/users/arichiardi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/arichiardi/subscriptions","organizations_url":"https://api.github.com/users/arichiardi/orgs","repos_url":"https://api.github.com/users/arichiardi/repos","events_url":"https://api.github.com/users/arichiardi/events{/privacy}","received_events_url":"https://api.github.com/users/arichiardi/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2016-06-10T22:56:34Z","updated_at":"2016-06-10T22:56:34Z","closed_at":null,"body":"Dear Drew, I am using successfully your library for building an email and I found out today while creating \r\n\r\n```clojure\r\n{:type :attachment\r\n :content-type (email-impl/guess-mime \"http://lorempixel.com/400/200/\" \"image/jpeg\")\r\n :content (io/as-url \"http://lorempixel.com/400/200/\")}\r\n```\r\n\r\nThrows a NPE:\r\n```\r\njava.lang.NullPointerException\r\n\tat javax.mail.internet.MimeUtility.quote(MimeUtility.java:927) ~[mailapi-1.4.3.jar:1.4.3]\r\n\tat javax.mail.internet.ParameterList.quote(ParameterList.java:649) ~[mailapi-1.4.3.jar:1.4.3]\r\n\tat javax.mail.internet.ParameterList.access$200(ParameterList.java:71) ~[mailapi-1.4.3.jar:1.4.3]\r\n\tat javax.mail.internet.ParameterList$ToStringBuffer.addNV(ParameterList.java:617) ~[mailapi-1.4.3.jar:1.4.3]\r\n\tat javax.mail.internet.ParameterList.toString(ParameterList.java:598) ~[mailapi-1.4.3.jar:1.4.3]\r\n\tat javax.mail.internet.ContentDisposition.toString(ContentDisposition.java:185) ~[mailapi-1.4.3.jar:1.4.3]\r\n\tat javax.mail.internet.MimeBodyPart.setFileName(MimeBodyPart.java:1161) ~[mailapi-1.4.3.jar:1.4.3]\r\n\tat javax.mail.internet.MimeBodyPart.setFileName(MimeBodyPart.java:531) ~[mailapi-1.4.3.jar:1.4.3]\r\n\tat postal.message$eval_bodypart.invokeStatic(message.clj:108) ~[?:?]\r\n\tat postal.message$eval_bodypart.invoke(message.clj:104) ~[?:?]\r\n\tat postal.message$eval54499$fn__54500.invoke(message.clj:98) ~[?:?]\r\n\tat postal.message$eval54482$fn__54483$G__54473__54488.invoke(message.clj:94) ~[?:?]\r\n\tat postal.message$eval_multipart.invokeStatic(message.clj:137) ~[?:?]\r\n\tat postal.message$eval_multipart.invoke(message.clj:127) ~[?:?]\r\n\tat postal.message$add_multipart_BANG_.invokeStatic(message.clj:141) ~[?:?]\r\n\tat postal.message$add_multipart_BANG_.invoke(message.clj:140) ~[?:?]\r\n\tat postal.message$add_body_BANG_.invokeStatic(message.clj:151) ~[?:?]\r\n\tat postal.message$add_body_BANG_.invoke(message.clj:148) ~[?:?]\r\n\tat postal.message$make_jmessage.invokeStatic(message.clj:177) ~[?:?]\r\n\tat postal.message$make_jmessage.invoke(message.clj:157) ~[?:?]\r\n\tat postal.message$make_jmessage.invokeStatic(message.clj:166) ~[?:?]\r\n\tat postal.message$make_jmessage.invoke(message.clj:157) ~[?:?]\r\n```\r\n\r\nI check the source and it fails [here](https://github.com/drewr/postal/blob/master/src/postal/message.clj#L110) because `re-find` returns `nil`. Probably a simple dummy file name would avoid this issue right? \r\n\r\nThanks!"}}
{"actor":{"display_login":"arichiardi"}}