secit-pl / imap-bundle

Simple php-imap integration for Symfony
MIT License
71 stars 20 forks source link

directory permissions wrong? #18

Closed akmez closed 2 years ago

akmez commented 2 years ago

created_attachments_dir_permissions is declared as an integer node in configuration https://github.com/secit-pl/imap-bundle/blob/master/DependencyInjection/Configuration.php#L53

When default value is 770 , the variable $directoryPermissions in https://github.com/secit-pl/imap-bundle/blob/master/Service/Imap.php#L119 has the integer value 770. And when we decoct(770) we have "1402" as permissions and creating the folder with wrong permissions.

Why dont pass the created_attachments_dir_permissions as scalarNode ?

php version 7.2.24 secit-pl/imap-bundle version 1.5.0

secit-pl commented 2 years ago

Should be octdec not decoct, sorry. Fixed in version 1.5.1.