Patch #30820

Drop setting "Blind carbon copy recipients (bcc)"

Added by Marius BALTEANU about 2 years ago. Updated 6 days ago.

Status:NewStart date:
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:Email notifications
Target version:5.0.0

Description

From #26791:

BCC recipients

Since each user gets their own mail, we can probably remove the setting to send mails with BCC completely. If plugins want to send mails to non-user recipients, we would have to make sure to either set the recipients manually on BCC or group them appropriately. It is not a required setting in Redmine core anymore though.

I agree, the setting doesn't make any sense now.

0001-Remove-setting-Blind-carbon-copy-recipients-bcc.patch Magnifier (23.8 KB) Marius BALTEANU, 2019-02-20 09:55

0001-Remove-setting-Blind-carbon-copy-recipients-bcc.patch Magnifier (25.9 KB) Marius BALTEANU, 2021-04-11 10:16

Related issues

Related to Redmine - Feature #26791: Send individual notification mails per mail recipient Closed
Related to Redmine - Defect #30787: Other recipients are not listed in To field even if Setti... Closed
Related to Redmine - Feature #8516: Email notifications without the recipient treated as atte... New 2011-06-03
Blocks Redmine - Patch #31035: Remove the workaround for #12090 New

History

#1 Updated by Marius BALTEANU about 2 years ago

  • Related to Feature #26791: Send individual notification mails per mail recipient added

#2 Updated by Marius BALTEANU about 2 years ago

  • Related to Defect #30787: Other recipients are not listed in To field even if Setting.bcc_recipients is false added

#3 Updated by Go MAEDA about 2 years ago

  • Assignee set to Jean-Philippe Lang
  • Target version changed from Candidate for next major release to 4.0.2

+1
The setting has no effect since #26791 was implemented.

#4 Updated by Go MAEDA about 2 years ago

Probably the patch also resolves #8516 because Redmine with the patch applied does not remove To field.

#5 Updated by Go MAEDA about 2 years ago

  • Related to Feature #8516: Email notifications without the recipient treated as attempted fraud by mail clients added

#6 Updated by Marius BALTEANU about 2 years ago

Go MAEDA wrote:

Probably the patch also resolves #8516 because Redmine with the patch applied does not remove To field.

Agree. Also, #30787 can be closed now from my point of view if you don't refer to other problem.

#7 Updated by Marius BALTEANU about 2 years ago

  • File deleted (0001-Remove-setting-Blind-carbon-copy-recipients-bcc.patch)

#8 Updated by Marius BALTEANU about 2 years ago

Updated to remove bcc_recipients from settings.yml as well.

#9 Updated by Go MAEDA about 2 years ago

  • Category set to Email notifications
  • Target version changed from 4.0.2 to 4.1.0

Since this patch adds a migration, I am changing the target version from 4.0.2 to 4.1.0.

#10 Updated by Go MAEDA about 2 years ago

  • Blocks Patch #31035: Remove the workaround for #12090 added

#11 Updated by Jean-Philippe Lang almost 2 years ago

  • Target version changed from 4.1.0 to 5.0.0

Removing a setting may cause some plugins to error.
Setting it to a major release.

#12 Updated by Marius BALTEANU 14 days ago

  • Assignee changed from Jean-Philippe Lang to Marius BALTEANU

#13 Updated by Marius BALTEANU 7 days ago

I've rebased the branch for the current trunk

#14 Updated by Marius BALTEANU 7 days ago

Marius BALTEANU wrote:

I've rebased the patch for the current trunk

#15 Updated by Holger Just 6 days ago

If entirely removing the setting is unwanted in 4.3, maybe we can deprecate it there.

That is, we could remove any code which uses the setting but keep the setting itself available in code (possible with some deprecation warning if accessed)

Also available in: Atom PDF