jro_apps/redmine Commit - r14308:3795c22730f4

Makes project selection by subaddress optional (#20732)....

Jean-Philippe Lang -

r14308:3795c22730f4

parent child

Context file:

r14308:3795c22730f4

Collapse all files

app/models/mail_handler.rb +6 -2

                  %w(project status tracker category priority).each do |option|
                    options[:issue][option.to_sym] = env[option] if env[option]
                  end
-                 %w(allow_override unknown_user no_permission_check no_account_notice default_group).each do |option|
+                 %w(allow_override unknown_user no_permission_check no_account_notice default_group project_from_subaddress).each do |option|
                    options[option.to_sym] = env[option] if env[option]
                  end
                  if env['private']
                end
                def get_project_from_receiver_addresses
+                 local, domain = handler_options[:project_from_subaddress].to_s.split("@")
+                 return nil unless local && domain
+                 local = Regexp.escape(local)
                  [:to, :cc, :bcc].each do |field|
                    header = @email[field]
                    next if header.blank? || header.field.blank? || !header.field.respond_to?(:addrs)
                    header.field.addrs.each do |addr|
-                     if addr.local.to_s =~ /\+([^+]+)\z/
+                     if addr.domain.to_s.casecmp(domain)==0 && addr.local.to_s =~ /\A#{local}\+([^+]+)\z/
                        if project = Project.find_by_identifier($1)
                          return project
                        end

extra/mail_handler/rdm-mailhandler.rb +20 -9

                VERSION = '0.2.3'
                attr_accessor :verbose, :issue_attributes, :allow_override, :unknown_user, :default_group, :no_permission_check,
-                 :url, :key, :no_check_certificate, :certificate_bundle, :no_account_notice, :no_notification
+                 :url, :key, :no_check_certificate, :certificate_bundle, :no_account_notice, :no_notification, :project_from_subaddress
                def initialize
                  self.issue_attributes = {}
                                                            "user") { |v| self.no_notification = '1'}
                    opts.separator("")
                    opts.separator("Issue attributes control options:")
+                   opts.on(      "--project-from-subaddress ADDR", "select project from subadress of ADDR found",
+                                                           "in To, Cc, Bcc headers") {|v| self.project_from_subaddress['project'] = v}
                    opts.on("-p", "--project PROJECT",      "identifier of the target project") {|v| self.issue_attributes['project'] = v}
                    opts.on("-s", "--status STATUS",        "name of the target status") {|v| self.issue_attributes['status'] = v}
                    opts.on("-t", "--tracker TRACKER",      "name of the target tracker") {|v| self.issue_attributes['tracker'] = v}
                * project, tracker, status, priority, category, assigned_to, fixed_version,
                  start_date, due_date, estimated_hours, done_ratio
                * custom fields names with underscores instead of spaces (case insensitive)
                Example: --allow_override=project,priority,my_custom_field
                If the --project option is not set, project is overridable by default for
                You can use --allow_override=all to allow all attributes to be overridable.
              Examples:
-               No project specified, emails MUST contain the 'Project' keyword:
-               rdm-mailhandler.rb --url http://redmine.domain.foo --key secret
+               No project specified, emails MUST contain the 'Project' keyword, otherwise
+               they will be dropped (not recommanded):
+                 rdm-mailhandler.rb --url http://redmine.domain.foo --key secret
                Fixed project and default tracker specified, but emails can override
                both tracker and priority attributes using keywords:
-               rdm-mailhandler.rb --url https://domain.foo/redmine --key secret \\
-                 --project foo \\
-                 --tracker bug \\
-                 --allow-override tracker,priority
+                 rdm-mailhandler.rb --url https://domain.foo/redmine --key secret \\
+                   --project myproject \\
+                   --tracker bug \\
+                   --allow-override tracker,priority
+               Project selected by subaddress of redmine@example.net. Sending the email
+               to redmine+myproject@example.net will add the issue to myproject:
+                 rdm-mailhandler.rb --url http://redmine.domain.foo --key secret \\
+                   --project-from-subaddress redmine@example.net
              END_DESC
                    opts.summary_width = 27
                                         'default_group' => default_group,
                                         'no_account_notice' => no_account_notice,
                                         'no_notification' => no_notification,
-                                        'no_permission_check' => no_permission_check}
+                                        'no_permission_check' => no_permission_check,
+                                        'project_from_subaddress' => project_from_subaddress}
                  issue_attributes.each { |attr, value| data["issue[#{attr}]"] = value }
                  debug "Posting to #{uri}..."

test/unit/mail_handler_test.rb +2 -1

                  # This email has redmine+onlinestore@somenet.foo as 'To' header
                  issue = submit_email(
                            'ticket_on_project_given_by_to_header.eml',
-                           :issue => {:tracker => 'Support request'}
+                           :issue => {:tracker => 'Support request'},
+                           :project_from_subaddress => 'redmine@somenet.foo'
                          )
                  assert issue.is_a?(Issue)
                  assert !issue.new_record?

General Comments 0

Write
Preview

You need to be logged in to leave comments. Login now

No TODOs yet

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages