Feature #94544: Allow more SMTP Configuration Settings (verify_peer) - TYPO3 Core - TYPO3 Forge

Custom queries

12 LTS
Accessibility
Dashboard issues
Easy tasks
Extbase Issues
FAL Issues
FAL Sprint 2023
Hierarchical Issues
Issues reported for v11
JavaScript issues
JavaScript Issues (w/o Epics/Stories)
Most wanted Bugfixes
Most wanted Features
My open issues
New & Unassigned
No feedback within 90 days
Open Bugs
Open issues of 10
Page Tree Regressions after 2020-07-28
Recently modified
Regressions
Stabilization Issues
Usability or UX

Watchers (1)

Ayke Halder

Actions

Copy link

Feature #94544

closed

Allow more SMTP Configuration Settings (verify_peer)

Added by varioous OG almost 3 years ago. Updated over 1 year ago.

Status:

Closed

Priority:

Should have

Assignee:

Category:

Mailer API

Target version:

Start date:

2021-07-12

Due date:

% Done:

100%

Estimated time:

PHP Version:

7.4

Tags:

Complexity:

Sprint Focus:

Description

Hi,

I need to send SMTP-Mails with ssl and a self signed certificate. In default configuration from Backend test mail I always receive:

Core: Error handler (BE): PHP Warning: stream_socket_enable_crypto(): SSL operation failed with code 1. OpenSSL Error messages: error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed in /data/www/httpdocs/releases/2021-06-28_19 47/vendor/symfony/mailer/Transport/Smtp/Stream/SocketStream.php line 155

Now I added in the \TYPO3\CMS\Core\Mail\TransportFactory the options to disable verfify check:

$stream = $transport->getStream();
$streamOptions = $stream->getStreamOptions();
$streamOptions['ssl']['verify_peer'] = false;
$streamOptions['ssl']['verify_peer_name'] = false;
$stream->setStreamOptions($streamOptions);

With this change sending works great.

I wish that can add this settings in LocalConfiguration Mail Settings

Related issues 2 (0 open — 2 closed)

Related to TYPO3 Core - Feature #96034: Add Symfony mailer configuration

Closed

2021-11-18

Actions

Related to TYPO3 Core - Bug #96803: Use proper default config values for new mail settings

Closed

Benni Mack

2022-02-09

Actions

Issue # Delay: days Cancel

History
Notes
Property changes
Associated revisions

Actions

Copy link

Updated by Guido Schmechel almost 3 years ago

I have a few questions about this:

- Where did you add these options? My guess is public function get(array $mailSettings) at the type "smtp"?
- Do you have an overview of all StreamOptions?

So that we can implement it in a meaningful way. I imagine an array in the AdditionalConfiguration, which is not editable by the Install Tool. Comparable to the HTTP options ([HTTP][proxy]). Here we have no UI on the part of the InstallTool.

Actions

Copy link

Updated by Gerrit Code Review almost 3 years ago

Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/69907

Actions

Copy link

Updated by Gerrit Code Review almost 3 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/69907

Actions

Copy link

Updated by Gerrit Code Review almost 3 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/69907

Actions

Copy link

Updated by Guido Schmechel almost 3 years ago

I add a first approach. Maybe you can check this patch and add some feedback?

Actions

Copy link

Updated by David Henninger over 2 years ago

I added the patch to my Typo3 10.4.21 setup and mail worked again after adding the new transport_smtp_stream_options to LocalConfiguration MAIL:

'MAIL' => [
    'transport'                     => 'smtp',
    'transport_sendmail_command'    => '/usr/sbin/sendmail -t -i ',
    'transport_smtp_encrypt'        => false,
    'transport_smtp_password'       => '',
    'transport_smtp_server'         => 'localhost:25',
    'transport_smtp_username'       => '',
    'transport_smtp_stream_options' => [
        'ssl' => [
            'verify_peer'      => false,
            'verify_peer_name' => false
        ]
    ]
]

However, the 'Configure Installation-Wide Options' in the install tool fails with the patch as "type: mixed" cannot handle the array above:

PHP Warning: mb_strlen() expects parameter 1 to be string, array given in typo3_src/typo3_src-10.4.21/typo3/sysext/frontend/Classes/ContentObject/ContentObjectRenderer.php line 3300

Also, some indentation spaces for the new options are missing in DefaultConfigurationDescription.yaml as well

P.S. yaml is the worst

Actions

Copy link

Updated by Gerrit Code Review over 2 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/69907

Actions

Copy link