Closed Bug 1576828 Opened 1 year ago Closed 4 months ago

Remove pref mailnews.p7m_subparts_external

Categories

(Thunderbird :: Security, task)

task
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 71.0

People

(Reporter: KaiE, Assigned: KaiE)

Details

Attachments

(1 file)

We should remove pref mailnews.p7m_subparts_external because setting it to the non-default value exposes the user to security risks.

We should no longer require the pref, because downloading of the attached messages is possible.

Attached patch 1576828-v1.patchSplinter Review
Assignee: nobody → kaie
Attachment #9101913 - Flags: review?(mkmelin+mozilla)
Attachment #9101913 - Flags: feedback?(giulia.duemari)
Comment on attachment 9101913 [details] [diff] [review]
1576828-v1.patch

Review of attachment 9101913 [details] [diff] [review]:
-----------------------------------------------------------------

Yeah as far as I know it's no longer needed.
Attachment #9101913 - Flags: review?(mkmelin+mozilla) → review+

TODO: needs a try build

Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/29569cac6cec
Remove pref mailnews.p7m_subparts_external and related code/tests. r=mkmelin DONTBUILD

Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED

Sorry, I went ahead and landed this after running the test locally. I don't expect any issues and I'm happy to wear the blame ;-)

Target Milestone: --- → Thunderbird 71.0

Sigh, I ran the linter to avoid linting issue but it spat the dummy

$ ../mach eslint mailnews/mime/
Exception in thread ProcessReaderStdout:
Traceback (most recent call last):
  File "c:\mozilla-build\python3\lib\threading.py", line 916, in _bootstrap_inner
    self.run()
  File "c:\mozilla-build\python3\lib\threading.py", line 864, in run
    self._target(*self._args, **self._kwargs)
  File "c:\mozilla-source\comm-central\testing/mozbase/mozprocess\mozprocess\processhandler.py", line 1030, in _read_stream
    line = stream.readline()
  File "c:\mozilla-build\python3\lib\encodings\cp1252.py", line 23, in decode
    return codecs.charmap_decode(input,self.errors,decoding_table)[0]
UnicodeDecodeError: 'charmap' codec can't decode byte 0x8f in position 3272: character maps to <undefined>

✖ 0 problems (0 errors, 0 warnings)

and lied to me since there are problems :-(

Anyway, since I don't have many patches to land, I'll do a real push with the fix in about 6 hours from now. (Edit: Changed my mind)

Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/5d026b6b005e
Follow-up: Fix linting issues. rs=white-space-only DONTBUILD

Sorry for the delay.

My patch was related to bug 243833 and the hidden preference used in the code is mailnew.p7m_external

So, i do not see any problem

Comment on attachment 9101913 [details] [diff] [review]
1576828-v1.patch

Review of attachment 9101913 [details] [diff] [review]:
-----------------------------------------------------------------

Sorry for the delay.

My patch was related to bug 243833 and the hidden preference used in the code is mailnew.p7m_external

So, i do not see any problem
Comment on attachment 9101913 [details] [diff] [review]
1576828-v1.patch

Review of attachment 9101913 [details] [diff] [review]:
-----------------------------------------------------------------

Sorry for the delay.

My patch was related to bug 243833 and the hidden preference used in the code is mailnew.p7m_external

So, i do not see any problem
You need to log in before you can comment on or make changes to this bug.