fix(smtp_server): fixes issue with malformed rcpt to

closes #1299 closes #1019
postalserver · Jul 27, 2021 · e0ba05a · e0ba05a
1 parent 73870d6
commit e0ba05a
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/lib/postal/smtp_server/client.rb b/lib/postal/smtp_server/client.rb
@@ -267,7 +267,17 @@ def rcpt_to(data)
         end
 
         rcpt_to = data.gsub(/RCPT TO\s*:\s*/i, '').gsub(/.*</, '').gsub(/>.*/, '').strip
+
+        if rcpt_to.blank?
+          return '501 RCPT TO should not be empty'
+        end
+
         uname, domain = rcpt_to.split('@', 2)
+
+        if domain.blank?
+          return '501 Invalid RCPT TO'
+        end
+
         uname, tag = uname.split('+', 2)
 
         if domain == Postal.config.dns.return_path || domain =~ /\A#{Regexp.escape(Postal.config.dns.custom_return_path_prefix)}\./