sisimai 4.25.4-java → 4.25.10-java

data/README.md

@@ -1,9 +1,9 @@
 ![](https://libsisimai.org/static/images/logo/sisimai-x01.png)
 
-[![License](https://img.shields.io/badge/license-BSD%202--Clause-orange.svg)](https://github.com/sisimai/rb-Sisimai/blob/master/LICENSE)
-[![Coverage Status](https://img.shields.io/coveralls/sisimai/rb-Sisimai.svg)](https://coveralls.io/r/sisimai/rb-Sisimai)
-[![Build Status](https://travis-ci.org/sisimai/rb-Sisimai.svg?branch=master)](https://travis-ci.org/sisimai/rb-Sisimai) 
-[![Codacy Badge](https://api.codacy.com/project/badge/grade/38340177e6284a65be69c0c7c3dc2b58)](https://www.codacy.com/app/azumakuniyuki/rb-Sisimai)
+[![License](https://img.shields.io/badge/license-BSD%202--Clause-orange.svg)](https://github.com/sisimai/rb-sisimai/blob/master/LICENSE)
+[![Coverage Status](https://img.shields.io/coveralls/sisimai/rb-sisimai.svg)](https://coveralls.io/r/sisimai/rb-sisimai)
+[![Build Status](https://travis-ci.org/sisimai/rb-sisimai.svg?branch=master)](https://travis-ci.org/sisimai/rb-sisimai) 
+[![Codacy Badge](https://api.codacy.com/project/badge/grade/38340177e6284a65be69c0c7c3dc2b58)](https://www.codacy.com/app/azumakuniyuki/rb-sisimai)
 [![Ruby](https://img.shields.io/badge/ruby-v2.1.0--v2.6.0-red.svg)](https://www.ruby-lang.org/)
 [![Gem Version](https://badge.fury.io/rb/sisimai.svg)](https://badge.fury.io/rb/sisimai)
 
@@ -19,7 +19,6 @@
 - [Usage](#usage)
     - [Basic usage](#basic-usage)
     - [Convert to JSON](#convert-to-json)
-    - [Read bounce object](#read-bounce-object)
     - [Callback feature](#callback-feature)
     - [One-Liner](#one-liner)
     - [Output example](#output-example)
@@ -40,7 +39,7 @@ What is Sisimai
 ===============================================================================
 Sisimai is a Ruby library for analyzing RFC5322 bounce emails and generating
 structured data from parsed results. The Ruby version of Sisimai is ported from
-the Perl version of Sisimai at [github.com/sisimai/p5-Sisimai](https://github.com/sisimai/p5-Sisimai/).
+the Perl version of Sisimai at [github.com/sisimai/p5-sisimai](https://github.com/sisimai/p5-sisimai/).
 
 ![](https://libsisimai.org/static/images/figure/sisimai-overview-1.png)
 
@@ -53,17 +52,14 @@ Key Features
   * git clone & make
 * __High Precision of Analysis__
   * 2 times higher than bounceHammer
-  * Support 24 known MTAs and 5 unknown MTAs
-  * Support 22 major MSPs(Mail Service Providers)
-  * Support 2 major Cloud Email Delivery Services(JSON format)
-    * **WILL BE REMOVED AT Sisimai 4.25.5**
+  * Support 68 MTAs/MDAs/ESPs
   * Support Feedback Loop Message(ARF)
   * Can detect 29 error reasons
 
 Command line demo
 -------------------------------------------------------------------------------
 The following screen shows a demonstration of Sisimai at the command line using
-Ruby(rb-Sisimai) and Perl(p5-Sisimai) version of Sisimai.
+Ruby(rb-sisimai) and Perl(p5-sisimai) version of Sisimai.
 ![](https://libsisimai.org/static/images/demo/sisimai-dump-01.gif)
 
 Setting Up Sisimai
@@ -86,10 +82,10 @@ Install
 
 ```shell
 $ sudo gem install sisimai
-Fetching: sisimai-4.25.1.gem (100%)
-Successfully installed sisimai-4.25.1
-Parsing documentation for sisimai-4.25.1
-Installing ri documentation for sisimai-4.25.1
+Fetching: sisimai-4.25.5.gem (100%)
+Successfully installed sisimai-4.25.5
+Parsing documentation for sisimai-4.25.5
+Installing ri documentation for sisimai-4.25.5
 Done installing documentation for sisimai after 6 seconds
 1 gem installed
 ```
@@ -98,15 +94,15 @@ Done installing documentation for sisimai after 6 seconds
 
 ```shell
 $ cd /usr/local/src
-$ git clone https://github.com/sisimai/rb-Sisimai.git
-$ cd ./rb-Sisimai
+$ git clone https://github.com/sisimai/rb-sisimai.git
+$ cd ./rb-sisimai
 $ sudo make depend install-from-local
 gem install bundle rake rspec coveralls
 ...
 4 gems installed
 bundle exec rake install
-sisimai 4.25.1 built to pkg/sisimai-4.25.1.gem.
-sisimai (4.25.1) installed.
+sisimai 4.25.5 built to pkg/sisimai-4.25.5.gem.
+sisimai (4.25.5) installed.
 ```
 
 Usage
@@ -115,7 +111,9 @@ Usage
 Basic usage
 -------------------------------------------------------------------------------
 `make()` method provides feature for getting parsed data from bounced email 
-messages like following.
+messages like following. Beginning with v4.25.6, new accessor `origin` which
+keeps the path to email file as a data source is available.
+
 
 ```ruby
 #! /usr/bin/env ruby
@@ -143,6 +141,7 @@ if v.is_a? Array
     puts e.deliverystatus       # 5.1.1
     puts e.replycode            # 550
     puts e.reason               # userunknown
+    puts e.origin               # /var/spool/bounce/Maildir/new/1740074341.eml
 
     h = e.damn                  # Convert to HASH
     j = e.dump('json')          # Convert to JSON string
@@ -164,30 +163,6 @@ puts Sisimai.dump('/path/to/mbox')  # or path to Maildir/
 puts Sisimai.dump('/path/to/mbox', delivered: true)
 ```
 
-Read bounce object
--------------------------------------------------------------------------------
-**THIS FEATURE WILL BE REMOVED AT SISIMAI 4.25.5**
-
-The way to read a bounce object retrived from Cloud Email Services as JSON using
-their API is the following code. This feature is available at between v4.20.0
-and **v4.25.5**.
-
-```ruby
-#! /usr/bin/env ruby
-require 'json'
-require 'sisimai'
-
-j = JSON.load('{"notificationType"=>"Bounce", "bounce"=>{"...') # JSON String
-v = Sisimai.make(j, input: 'json')
-
-if v.is_a? Array
-  v.each do |e|
-    ...
-  end
-end
-```
-As of present, Only Amazon SES and SendGrid are supported.
-
 Callback feature
 -------------------------------------------------------------------------------
 Beginning with Sisimai 4.19.0, `make()` and `dump()` methods of Sisimai accept
@@ -207,9 +182,8 @@ callbackto = lambda do |v|
   return r
 end
 
-list = ['X-Mailer']
-data = Sisimai.make('/path/to/mbox', hook: callbackto, field: list)
-json = Sisimai.dump('/path/to/mbox', hook: callbackto, field: list)
+data = Sisimai.make('/path/to/mbox', hook: callbackto)
+json = Sisimai.dump('/path/to/mbox', hook: callbackto)
 
 puts data[0].catch['x-mailer']      # Apple Mail (2.1283)
 ```
@@ -230,7 +204,7 @@ Output example
 ![](https://libsisimai.org/static/images/demo/sisimai-dump-02.gif)
 
 ```json
-[{"action": "failed", "subject": "Nyaan", "catch": null, "token": "08acf78323edc7923a783c04749dd547ab45c433", "alias": "", "messageid": "201806090556.w595u8GZ093276@neko.example.jp", "listid": "", "smtpcommand": "MAIL", "smtpagent": "Email::Sendmail", "lhost": "localhost", "timezoneoffset": "+0900", "feedbacktype": "", "senderdomain": "neko.example.jp", "diagnostictype": "SMTP", "softbounce": 1, "deliverystatus": "5.0.0", "addresser": "kijitora@neko.example.jp", "diagnosticcode": "550 Unauthenticated senders not allowed", "timestamp": 1528523769, "destination": "example.com", "recipient": "sironeko@example.com", "reason": "securityerror", "replycode": "550", "rhost": "neko.example.jp"}]
+[{"catch":{"x-mailer":"","return-path":"<shironeko@mx.example.co.jp>"},"token":"cf17945938502bd876603a375f0e9517c921bbab","lhost":"localhost","rhost":"mx-s.neko.example.jp","alias":"","listid":"","reason":"hasmoved","action":"failed","origin":"set-of-emails/maildir/bsd/lhost-sendmail-22.eml","subject":"Nyaaaan","messageid":"0000000011111.fff0000000003@mx.example.co.jp","replycode":"","smtpagent":"Sendmail","softbounce":0,"smtpcommand":"DATA","destination":"example.net","senderdomain":"example.co.jp","feedbacktype":"","diagnosticcode":"450 busy - please try later 551 not our customer 503 need RCPT command [data]","diagnostictype":"SMTP","deliverystatus":"5.1.6","timezoneoffset":"+0900","addresser":"shironeko@example.co.jp","recipient":"kijitora@example.net","timestamp":1397054085}]
 ```
 
 Sisimai Specification
@@ -248,15 +222,15 @@ and bounceHammer are available at
 | System requirements                         | Ruby 2.1 - 2.6 | Perl 5.10 -   |
 |                                             | JRuby 9.0.4.0- |               |
 | Analytical precision ratio(2000 emails)[1]  | 1.00           | 1.00          |
-| The speed of parsing email(1000 emails)     | 2.22s[2]       | 1.50s         |
+| The speed of parsing email(1000 emails)     | 2.22s[2]       | 1.35s         |
 | How to install                              | gem install    | cpanm, cpm    |
 | Dependencies (Except core modules)          | 1 module       | 2 modules     |
-| LOC:Source lines of code                    | 11700 lines    | 8300 lines    |
-| The number of tests(spec/,t/,xt/) directory | 227600 tests   | 250000 tests  |
+| LOC:Source lines of code                    | 10300 lines    | 10500 lines   |
+| The number of tests(spec/,t/,xt/) directory | 236000 tests   | 265000 tests  |
 | License                                     | BSD 2-Clause   | BSD 2-Clause  |
 | Support Contract provided by Developer      | Available      | Available     |
 
-1. See [./ANALYTICAL-PRECISION](https://github.com/sisimai/rb-Sisimai/blob/master/ANALYTICAL-PRECISION)
+1. See [./ANALYTICAL-PRECISION](https://github.com/sisimai/rb-sisimai/blob/master/ANALYTICAL-PRECISION)
 2. Xeon E5-2640 2.5GHz x 2 cores | 5000 bogomips | 1GB RAM | Ruby 2.3.4p301
 
 Other specification of Sisimai
@@ -270,7 +244,7 @@ Contributing
 
 Bug report
 -------------------------------------------------------------------------------
-Please use the [issue tracker](https://github.com/sisimai/rb-Sisimai/issues)
+Please use the [issue tracker](https://github.com/sisimai/rb-sisimai/issues)
 to report any bugs.
 
 Emails could not be parsed
@@ -289,14 +263,14 @@ Related Sites
 * __libSISIMAI.ORG__ | [Sisimai | The successor to bounceHammer, Library to parse bounce mails](https://libsisimai.org/)
 * __Sisimai Blog__ | [blog.libsisimai.org](http://blog.libsisimai.org/)
 * __Facebook Page__ | [facebook.com/libsisimai](https://www.facebook.com/libsisimai/)
-* __GitHub__ | [github.com/sisimai/rb-Sisimai](https://github.com/sisimai/rb-Sisimai)
+* __GitHub__ | [github.com/sisimai/rb-sisimai](https://github.com/sisimai/rb-sisimai)
 * __RubyGems.org__ | [rubygems.org/gems/sisimai](https://rubygems.org/gems/sisimai)
-* __Perl verson__ | [Perl version of Sisimai](https://github.com/sisimai/p5-Sisimai)
+* __Perl verson__ | [Perl version of Sisimai](https://github.com/sisimai/p5-sisimai)
 * __Fixtures__ | [set-of-emails - Sample emails for "make test"](https://github.com/sisimai/set-of-emails)
 
 See also
 -------------------------------------------------------------------------------
-* [README-JA.md - README.md in Japanese(日本語)](https://github.com/sisimai/rb-Sisimai/blob/master/README-JA.md)
+* [README-JA.md - README.md in Japanese(日本語)](https://github.com/sisimai/rb-sisimai/blob/master/README-JA.md)
 * [RFC3463 - Enhanced Mail System Status Codes](https://tools.ietf.org/html/rfc3463)
 * [RFC3464 - An Extensible Message Format for Delivery Status Notifications](https://tools.ietf.org/html/rfc3464)
 * [RFC3834 - Recommendations for Automatic Responses to Electronic Mail](https://tools.ietf.org/html/rfc3834)
@@ -309,7 +283,7 @@ Author
 
 Copyright
 ===============================================================================
-Copyright (C) 2015-2019 azumakuniyuki, All Rights Reserved.
+Copyright (C) 2015-2020 azumakuniyuki, All Rights Reserved.
 
 License
 ===============================================================================

data/Repository.mk

@@ -1,4 +1,4 @@
-# rb-Sisimai/Repository.mk
+# rb-sisimai/Repository.mk
 #  ____                      _ _                               _    
 # |  _ \ ___ _ __   ___  ___(_) |_ ___  _ __ _   _   _ __ ___ | | __
 # | |_) / _ \ '_ \ / _ \/ __| | __/ _ \| '__| | | | | '_ ` _ \| |/ /

data/lib/sisimai.rb

@@ -1,10 +1,9 @@
-require 'sisimai/version'
-
 # Sisimai is the system formerly known as bounceHammer 4, is a Ruby module for
 # analyzing bounce mails and generate structured data in a JSON format (YAML is
 # also available if "YAML" module is installed on your system) from parsed bounce
 # messages. Sisimai is a coined word: Sisi (the number 4 is pronounced "Si" in
 # Japanese) and MAI (acronym of "Mail Analyzing Interface").
+require 'sisimai/version'
 module Sisimai
   # Imported from p5-Sisimail/lib/Sisimai.pm
   class << self
@@ -19,82 +18,29 @@ module Sisimai
     # @param         [Hash]   argv1      Parser options(delivered=false)
     # @options argv1 [Boolean] delivered true: Include "delivered" reason
     # @options argv1 [Lambda]  hook      Lambda object to be called back
-    # @options argv1 [String]  input     Input data format: 'email', 'json'
-    # @options argv1 [Array]   field     Email header name to be captured
     # @return        [Array]             Parsed objects
     # @return        [nil]               nil if the argument was wrong or an empty array
     def make(argv0, **argv1)
       return nil unless argv0
-
-      input = argv1[:input] || nil
-      field = argv1[:field] || []
-      raise ' ***error: "field" accepts an array only' unless field.is_a? Array
-
-      unless input
-        klass = argv0.class
-        # "input" did not specified, try to detect automatically.
-        if klass == ::String || klass == IO
-          # The argument may be a path to email OR an email text
-          input = 'email'
-        elsif klass == Array || klass == Hash
-          # The argument may be a decoded JSON object
-          input = 'json'
-        end
-      end
-
-      delivered1 = argv1[:delivered] || false
-      hookmethod = argv1[:hook] || nil
-      bouncedata = []
-
       require 'sisimai/data'
       require 'sisimai/message'
-      if input == 'email'
-        # Path to mailbox or Maildir/, or STDIN: 'input' => 'email'
-        require 'sisimai/mail'
-        return nil unless mail = Sisimai::Mail.new(argv0)
-
-        while r = mail.read do
-          # Read and parse each mail file
-          methodargv = { data: r, hook: hookmethod, input: 'email', field: field }
-          mesg = Sisimai::Message.new(methodargv)
-          next if mesg.void
-
-          methodargv = { data: mesg, hook: hookmethod, input: 'email', delivered: delivered1 }
-          next unless data = Sisimai::Data.make(methodargv)
-          bouncedata += data unless data.empty?
-        end
-
-      elsif input == 'json'
-        # Decoded JSON object: 'input' => 'json'
-        warn ' ***warning: input: "json" is marked as obsoleted'
-        type = argv0.class.to_s
-        list = []
-
-        if type == 'Array'
-          # [ {...}, {...}, ... ]
-          while e = argv0.shift do
-            list << e
-          end
-        else
-          list << argv0
-        end
-
-        while e = list.shift do
-          methodargv = { data: e, hook: hookmethod, input: 'json' }
-          mesg = Sisimai::Message.new(methodargv)
-          next if mesg.void
-
-          methodargv = { data: mesg, hook: hookmethod, input: 'json', delivered: delivered1 }
-          next unless data = Sisimai::Data.make(methodargv)
-          bouncedata += data unless data.empty?
-        end
-      else
-        # The value of "input" neither "email" nor "json"
-        raise ' ***error: invalid value of "input"'
+      require 'sisimai/mail'
+
+      list = []
+      return nil unless mail = Sisimai::Mail.new(argv0)
+      while r = mail.data.read do
+        # Read and parse each email file
+        methodargv = { data: r, hook: argv1[:hook] }
+        mesg = Sisimai::Message.new(**methodargv)
+        next if mesg.void
+
+        methodargv = { data: mesg, delivered: argv1[:delivered], origin: mail.data.path }
+        next unless data = Sisimai::Data.make(**methodargv)
+        list += data unless data.empty?
       end
 
-      return nil if bouncedata.empty?
-      return bouncedata
+      return nil if list.empty?
+      return list 
     end
 
     # Wrapper method to parse mailbox/Maildir and dump as JSON
@@ -104,12 +50,11 @@ module Sisimai
     # @param         [Hash] argv1        Parser options
     # @options argv1 [Integer] delivered true: Include "delivered" reason
     # @options argv1 [Lambda]  hook      Lambda object to be called back
-    # @options argv1 [String]  input     Input data format: 'email', 'json'
     # @return        [String]            Parsed data as JSON text
     def dump(argv0, **argv1)
       return nil unless argv0
 
-      nyaan = Sisimai.make(argv0, argv1) || []
+      nyaan = Sisimai.make(argv0, **argv1) || []
       if RUBY_PLATFORM.start_with?('java')
         # java-based ruby environment like JRuby.
         require 'jrjackson'

data/lib/sisimai/address.rb

@@ -30,7 +30,7 @@ module Sisimai
     #           # => Sisimai::Address object
     def self.make(argvs)
       return nil unless argvs.is_a? Hash
-      return nil unless argvs.key?(:address)
+      return nil unless argvs[:address]
       return nil if argvs[:address].empty?
 
       thing = Sisimai::Address.new(argvs[:address])
@@ -69,7 +69,7 @@ module Sisimai
 
       while e = characters.shift do
         # Check each characters
-        if Delimiters.key?(e)
+        if Delimiters[e]
           # The character is a delimiter character
           if e == ','
             # Separator of email addresses or not
@@ -283,7 +283,7 @@ module Sisimai
     #   expand_verp('bounce+neko=example.org@example.org') #=> 'neko@example.org'
     def self.expand_verp(email)
       return nil unless email.is_a? Object::String
-      return nil unless cv = email.split('@', 2).first.match(/\A[-_\w]+?[+](\w[-._\w]+\w)[=](\w[-.\w]+\w)\z/)
+      return nil unless cv = email.split('@', 2).first.match(/\A[-\w]+?[+](\w[-.\w]+\w)[=](\w[-.\w]+\w)\z/)
       verp0 = cv[1] + '@' + cv[2]
       return verp0 if Sisimai::RFC5322.is_emailaddress(verp0)
     end
@@ -297,23 +297,19 @@ module Sisimai
       return nil unless Sisimai::RFC5322.is_emailaddress(email)
 
       local = email.split('@')
-      return nil unless cv = local[0].match(/\A([-_\w]+?)[+].+\z/)
+      return nil unless cv = local[0].match(/\A([-\w]+?)[+].+\z/)
       return cv[1] + '@' + local[1]
     end
 
-    @@roaccessors = [
-      :address, # [String] Email address
-      :user,    # [String] local part of the email address
-      :host,    # [String] domain part of the email address
-      :verp,    # [String] VERP
-      :alias,   # [String] alias of the email address
-    ]
-    @@rwaccessors = [
-      :name,    # [String] Display name
-      :comment, # [String] Comment
-    ]
-    @@roaccessors.each { |e| attr_reader   e }
-    @@rwaccessors.each { |e| attr_accessor e }
+    # :address, # [String] Email address
+    # :user,    # [String] local part of the email address
+    # :host,    # [String] domain part of the email address
+    # :verp,    # [String] VERP
+    # :alias,   # [String] alias of the email address
+    # :name,    # [String] Display name
+    # :comment, # [String] Comment
+    attr_reader   :address, :user, :host, :verp, :alias
+    attr_accessor :name, :comment
 
     # Constructor of Sisimai::Address
     # @param <str>  [String] argv1          Email address

data/lib/sisimai/arf.rb

@@ -14,28 +14,31 @@ module Sisimai
       # OpenDMARC 1.3.0 uses:  This is an authentication failure report for an email message received from IP
       # Abusix ARF uses        this is an autogenerated email abuse complaint regarding your network.
       Indicators = Sisimai::Lhost.INDICATORS
-      StartingOf = { rfc822: ['Content-Type: message/rfc822', 'Content-Type: text/rfc822-headers'] }.freeze
+      StartingOf = {
+        rfc822: ['Content-Type: message/rfc822', 'Content-Type: text/rfc822-headers'],
+        report: ['Content-Type: message/feedback-report'],
+      }.freeze
       MarkingsOf = {
         message: %r{\A(?>
-           [Tt]his[ ]is[ ].+[ ]email[ ]abuse[ ]report
-          |[Tt]his[ ]is[ ](?:
-             an[ ]autogenerated[ ]email[ ]abuse[ ]complaint
-            |an?[ ].+[ ]report[ ]for
-            |a[ ].+[ ]authentication[ -]failure[ ]report[ ]for
-            )
+          [Tt]his[ ]is[ ]a[ ][^ ]+[ ](?:email[ ])?[Aa]buse[ ][Rr]eport
+         |[Tt]his[ ]is[ ]an[ ]email[ ]abuse[ ]report
+         |[Tt]his[ ]is[ ](?:
+            a[ ][^ ]+[ ]authentication[ -]failure[ ]report
+           |an[ ]authentication[ -]failure[ ]report
+           |an[ ]autogenerated[ ]email[ ]abuse[ ]complaint
+           |an?[ ][^ ]+[ ]report[ ]for
+           )
           )
         }x,
       }.freeze
+      ReportFrom = /(?:staff[@]hotmail[.]com|complaints[@]email-abuse[.]amazonses[.]com)\z/.freeze
       LongFields = Sisimai::RFC5322.LONGFIELDS
       RFC822Head = Sisimai::RFC5322.HEADERFIELDS
 
       def description; return 'Abuse Feedback Reporting Format'; end
-      def smtpagent;   return 'FeedBack-Loop'; end
-      def headerlist;  return []; end
 
       # Email is a Feedback-Loop message or not
-      # @param    [Hash] heads    Email header including "Content-Type", "From",
-      #                           and "Subject" field
+      # @param    [Hash] heads    Email header including "Content-Type", "From", and "Subject" field
       # @return   [True,False]    true: Feedback Loop
       #                           false: is not Feedback loop
       def is_arf(heads)
@@ -49,13 +52,8 @@ module Sisimai
         elsif heads['content-type'].start_with?('multipart/mixed')
           # Microsoft (Hotmail, MSN, Live, Outlook) uses its own report format.
           # Amazon SES Complaints bounces
-          hfrom = Sisimai::Address.s3s4(heads['from'])
-          mfrom = %r{(?:
-             staff[@]hotmail[.]com
-            |complaints[@]email-abuse[.]amazonses[.]com
-            )\z
-          }x
-          if hfrom =~ mfrom && heads['subject'].include?('complaint about message from ')
+          p = Sisimai::Address.s3s4(heads['from'])
+          if p =~ ReportFrom && heads['subject'].include?('complaint about message from ')
             # From: staff@hotmail.com
             # From: complaints@email-abuse.amazonses.com
             # Subject: complaint about message from 192.0.2.1
@@ -66,21 +64,15 @@ module Sisimai
       end
 
       # Detect an error for Feedback Loop
-      # @param         [Hash] mhead       Message headers of a bounce email
-      # @options mhead [String] from      From header
-      # @options mhead [String] date      Date header
-      # @options mhead [String] subject   Subject header
-      # @options mhead [Array]  received  Received headers
-      # @options mhead [String] others    Other required headers
-      # @param         [String] mbody     Message body of a bounce email
-      # @return        [Hash, Nil]        Bounce data list and message/rfc822 part
-      #                                   or nil if it failed to parse or the
-      #                                   arguments are missing
+      # @param  [Hash] mhead    Message headers of a bounce email
+      # @param  [String] mbody  Message body of a bounce email
+      # @return [Hash]          Bounce data list and message/rfc822 part
+      # @return [Nil]           it failed to parse or the arguments are missing
       def make(mhead, mbody)
         return nil unless self.is_arf(mhead)
 
         dscontents = [Sisimai::Lhost.DELIVERYSTATUS]
-        hasdivided = mbody.split("\n")
+        bodyslices = mbody.split("\n")
         rfc822part = ''   # (String) message/rfc822-headers part
         previousfn = ''   # (String) Previous field name
         readcursor = 0    # (Integer) Points the current cursor position
@@ -120,13 +112,18 @@ module Sisimai
         #      generator is using to generate the report.  The version number in
         #      this specification is set to "1".
         #
-        while e = hasdivided.shift do
+        while e = bodyslices.shift do
+          # This is an email abuse report for an email message with the
+          #   message-id of 0000-000000000000000000000000000000000@mx
+          #   received from IP address 192.0.2.1 on
+          #   Thu, 29 Apr 2010 00:00:00 +0900 (JST)
+          if e =~ MarkingsOf[:message]
+            commondata['diagnosis'] = e if commondata['diagnosis'].empty?
+          end
+
           if readcursor == 0
             # Beginning of the bounce message or message/delivery-status part
-            if e =~ MarkingsOf[:message]
-              readcursor |= Indicators[:deliverystatus]
-              next
-            end
+            readcursor |= Indicators[:deliverystatus] if e.start_with?(StartingOf[:report][0])
           end
 
           if (readcursor & Indicators[:'message-rfc822']) == 0
@@ -153,11 +150,18 @@ module Sisimai
             elsif cv = e.match(/\AFrom:[ ]*(.+)\z/)
               # Microsoft ARF: original sender.
               commondata['from'] = Sisimai::Address.s3s4(cv[1]) if commondata['from'].empty?
+              previousfn = 'from'
 
             elsif e.start_with?(' ', "\t")
               # Continued line from the previous line
-              rfc822part << e + "\n" if LongFields.key?(previousfn)
-              next unless e.empty?
+              if previousfn == 'from'
+                # Multiple lines at "From:" field
+                commondata['from'] << e
+                next
+              else
+                rfc822part << e + "\n" if LongFields[previousfn]
+                next unless e.empty?
+              end
               rcptintext << e if previousfn == 'to'
 
             else
@@ -167,14 +171,14 @@ module Sisimai
               lhs.downcase!
 
               previousfn = ''
-              next unless RFC822Head.key?(lhs)
+              next unless RFC822Head[lhs]
 
               previousfn  = lhs
               rfc822part << e + "\n"
               rcptintext  = rhs if lhs == 'to'
             end
           else
-            # message/delivery-status part
+            # message/feedback-report part
             next unless readcursor & Indicators[:deliverystatus] > 0
             next if e.empty?
 
@@ -231,7 +235,7 @@ module Sisimai
               # Reporting-MTA: dns; mx.example.jp
               commondata['rhost'] = cv[1]
 
-            elsif cv = e.match(/\ASource-IP:[ ]*(.+)\z/)
+            elsif cv = e.match(/\ASource-I[Pp]:[ ]*(.+)\z/)
               # The header is optional and MUST NOT appear more than once.
               # Source-IP: 192.0.2.45
               arfheaders['rhost'] = cv[1]
@@ -241,12 +245,6 @@ module Sisimai
               # Original-Mail-From: <somespammer@example.net>
               commondata['from'] = Sisimai::Address.s3s4(cv[1]) if commondata['from'].empty?
 
-            elsif e =~ MarkingsOf[:message]
-              # This is an email abuse report for an email message with the
-              #   message-id of 0000-000000000000000000000000000000000@mx
-              #   received from IP address 192.0.2.1 on
-              #   Thu, 29 Apr 2010 00:00:00 +0900 (JST)
-              commondata['diagnosis'] = e
             end
           end
         end
@@ -292,9 +290,10 @@ module Sisimai
 
           e['softbounce'] = -1
           e['diagnosis']  = commondata['diagnosis'] unless e['diagnosis']
+          e['diagnosis']  = Sisimai::String.sweep(e['diagnosis'])
           e['date']       = mhead['date'] if e['date'].empty?
           e['reason']     = 'feedback'
-          e['agent']      = self.smtpagent if e['agent'].empty?
+          e['agent']      = 'Feedback-Loop'
           %w[command action status alias].each { |a| e[a] = '' }
 
           # Get the remote IP address from the message body
@@ -308,7 +307,6 @@ module Sisimai
             # from IP address 24.64.1.1 on Thu, 29 Apr 2010 00:00:00 +0000
             e['rhost'] = cv[1]
           end
-
         end
         return { 'ds' => dscontents, 'rfc822' => rfc822part }
       end