This is a notice to the maintainers of this module from the LWG about a violation of the Drupal.org Drupal Git Repository Usage policy.

SMTP Authentication Support comes bundled with the files PHPMailer.php and SMTP.php from PHPMailer. (https://github.com/PHPMailer/PHPMailer), These files are third party PHP code Copyright 2001-2003, Brent R. Matzelle, 2004-2009, Andy Prevost. . All Rights Reserved.

Third party code/assets is not generally allowed on Drupal.org and should be deleted from the repo.

This policy is described in the 3rd party libraries and content on Drupal.org. It also appears in the Drupal Git Repository Usage policy you agreed to when you signed up for Git access, which you may want to re-read, to be sure you're not violating other terms.

The Libraries API module is a recommended method for adding 3rd party dependencies without directly including the code on Drupal.org.

I am aware that concern about 3rd party code as been raised before (see related issues), but it has not been resolved, as it has been argued that this only applies to "code libraries". I don't think it was ever the case, but now, the 3rd party libraries and content on Drupal.org is very clear:

To avoid any misunderstanding: This policy applies all third party materials, including code (PHP and Javascript), CSS, media files (fonts, images, videos and sound) and text.

For more information, please see Contacted by the LWG. Now what?.

Issue tracker in the LWG issue queue: #2758911: 3rd party PHP code in SMTP Authentication Support.

This notice is also sent to the project owner (José San Martin) and active maintainers (Fabiano Sant'Ana, yettyn) through the Drupal.org contact form.

Comments

gisle created an issue. See original summary.

Ramdas Gaikar’s picture

Hello,

This can be accomplished by two way :

1) Creating composer.json file for this module.
2) include smtp.libraries.yml for this module.

phpmailer:
  remote: https://github.com/PHPMailer/PHPMailer
  version: 5.2.16
  php:
    PHPMailerAutoload.php: {}

I am not very good at Drupal - 8, so I think here Drupal - 8 geeks can help in better way.

Thanks.

pazhyn’s picture

Chris Matthews’s picture

Status: Active » Closed (duplicate)