-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Setup page requires a mail username #3244
Comments
We can make the username optional. It's just validation. For the time being, update the |
nalysius
added a commit
to nalysius/Cachet
that referenced
this issue
Oct 20, 2018
During the setup process email informations were asked: - Driver - Host - Username - Password In some situations the username is not useful because the Cachet's host may be configured to forward email to a server. The problem is the username was required, so we had to set a username and then update the .env file to remove it. To fix this problem, the mail username has been set to optional in the setup. So if someone needs a username it still can use this field, and otherwise people can let it empty. See: cachethq#3244
nalysius
added a commit
to nalysius/Cachet
that referenced
this issue
Dec 28, 2018
During the setup process email informations were asked: - Driver - Host - Username - Password In some situations the username is not useful because the Cachet's host may be configured to forward email to a server. The problem is the username was required, so we had to set a username and then update the .env file to remove it. To fix this problem, the mail username has been set to optional in the setup. So if someone needs a username it still can use this field, and otherwise people can let it empty. See: cachethq#3244
nalysius
added a commit
to nalysius/Cachet
that referenced
this issue
Dec 28, 2018
During the setup the "mail_username" was required and it was then undone, so using the sendmail driver we can let the username empty. It would be bad to let the username optional for every drivers, because in some configurations, like SMTP, the username is required for the SMTP server so if the user let it empty its mail configuration will be bad. The mail_username is now optional only if the mail driver is sendmail. See: cachethq#3244
Fixed in 6a53e21 |
zhenyulin
pushed a commit
to zhenyulin/Cachet
that referenced
this issue
Mar 12, 2019
* New translations cachet.php (Portuguese) * New translations cachet.php (Polish) * New translations cachet.php (Romanian) * New translations cachet.php (Japanese) * New translations cachet.php (Korean) * New translations cachet.php (Norwegian) * New translations forms.php (Norwegian) * New translations cachet.php (Persian) * New translations cachet.php (Russian) * New translations cachet.php (Ukrainian) * New translations cachet.php (Vietnamese) * New translations cachet.php (Zulu) * New translations forms.php (Zulu) * New translations cachet.php (Estonian) * New translations cachet.php (Spanish) * New translations cachet.php (Swedish) * New translations cachet.php (Thai) * New translations cachet.php (Turkish) * New translations cachet.php (Chinese Traditional) * New translations cachet.php (Czech) * New translations cachet.php (Danish) * New translations cachet.php (Dutch) * New translations forms.php (Dutch) * New translations forms.php (Chinese Simplified) * New translations cachet.php (Arabic) * New translations cachet.php (Albanian) * New translations cachet.php (Catalan) * New translations cachet.php (Chinese Simplified) * New translations cachet.php (Hebrew) * New translations cachet.php (Hungarian) * New translations cachet.php (Indonesian) * New translations cachet.php (Italian) * New translations cachet.php (Greek) * New translations cachet.php (English) * New translations cachet.php (Finnish) * New translations cachet.php (French) * New translations cachet.php (German) * Fix configuring postgres leads to wrong database driver key * Add warning when setting up localhost with mysql driver * Explicitly set the port number when not using traditional ports * Add support for empty values in the persistEnv method * [Security] Bump url-parse from 1.4.1 to 1.4.4 Bumps [url-parse](https://github.com/unshiftio/url-parse) from 1.4.1 to 1.4.4. **This update includes security fixes.** - [Release notes](https://github.com/unshiftio/url-parse/releases) - [Commits](unshiftio/url-parse@1.4.1...1.4.4) Signed-off-by: dependabot[bot] <support@dependabot.com> * Use PHP Debugger(phpdbg) for PHPUnit to speed up CI * Improve database performance by removing duplicated queries * StyleCI * Eager load user on the incident page not to query twice * Eager load group on the dashboard/components page not to query twice * Improve database performance by removing duplicated queries * StyleCI * Resolve cachethq#3269 * Resolve cachethq#3288 If Binput::get('enabled') is not defined this will return NULL which resulted in a false. Updated tests * New translations cachet.php (Swedish) * New translations dashboard.php (Swedish) * Apply fixes from StyleCI * Update deps * Drop support for RSS & Atom feeds. * Update Blade files * Don't expose current mail password * Rebuilt composer.lock * Bump laravolt/avatar from 1.8.1 to 2.1.0 Bumps [laravolt/avatar](https://github.com/laravolt/avatar) from 1.8.1 to 2.1.0. - [Release notes](https://github.com/laravolt/avatar/releases) - [Changelog](https://github.com/laravolt/avatar/blob/master/CHANGELOG.md) - [Commits](laravolt/avatar@1.8.1...2.1.0) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump laravel-mix-purgecss from 2.2.0 to 3.0.0 Bumps [laravel-mix-purgecss](https://github.com/spatie/laravel-mix-purgecss) from 2.2.0 to 3.0.0. - [Release notes](https://github.com/spatie/laravel-mix-purgecss/releases) - [Changelog](https://github.com/spatie/laravel-mix-purgecss/blob/master/CHANGELOG.md) - [Commits](spatie/laravel-mix-purgecss@2.2.0...3.0.0) Signed-off-by: dependabot[bot] <support@dependabot.com> * Upgraded JS deps and rebuilt assets * GitHub now supports transferring an issue * Remove unused $config property * Apply fixes from StyleCI * Upgraded to Laravel 5.7 * Deal with settings read/write errors properly * Apply fixes from StyleCI * Commit new vendor files * Apply fixes from StyleCI * Fixed up vendor views * Updated doctrine and emoji packages * Set the mail username optional in setup During the setup process email informations were asked: - Driver - Host - Username - Password In some situations the username is not useful because the Cachet's host may be configured to forward email to a server. The problem is the username was required, so we had to set a username and then update the .env file to remove it. To fix this problem, the mail username has been set to optional in the setup. So if someone needs a username it still can use this field, and otherwise people can let it empty. See: cachethq#3244 * Let the mail_username required except for sendmail During the setup the "mail_username" was required and it was then undone, so using the sendmail driver we can let the username empty. It would be bad to let the username optional for every drivers, because in some configurations, like SMTP, the username is required for the SMTP server so if the user let it empty its mail configuration will be bad. The mail_username is now optional only if the mail driver is sendmail. See: cachethq#3244 * Apply fixes from StyleCI * Upgraded dependencies following XSS vulnerability discovery * Use safer markdown defaults * [Security] Bump twig/twig from 1.36.0 to 2.6.0 Bumps [twig/twig](https://github.com/twigphp/Twig) from 1.36.0 to 2.6.0. **This update includes security fixes.** - [Release notes](https://github.com/twigphp/Twig/releases) - [Changelog](https://github.com/twigphp/Twig/blob/2.x/CHANGELOG) - [Commits](twigphp/Twig@v1.36.0...v2.6.0) Signed-off-by: dependabot[bot] <support@dependabot.com> * New translations forms.php (Dutch) * New translations cachet.php (German) * New translations pagination.php (German) * Apply fixes from StyleCI * Bump doctrine/dbal from 2.9.1 to 2.9.2 Bumps [doctrine/dbal](https://github.com/doctrine/dbal) from 2.9.1 to 2.9.2. - [Release notes](https://github.com/doctrine/dbal/releases) - [Commits](doctrine/dbal@v2.9.1...v2.9.2) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump laravel-mix and laravel-mix-purgecss Bumps [laravel-mix](https://github.com/JeffreyWay/laravel-mix) and [laravel-mix-purgecss](https://github.com/spatie/laravel-mix-purgecss). These dependencies needed to be updated together. Updates `laravel-mix` from 2.1.14 to 4.0.12 - [Release notes](https://github.com/JeffreyWay/laravel-mix/releases) - [Commits](laravel-mix/laravel-mix@v2.1.14...v4.0.12) Updates `laravel-mix-purgecss` from 3.0.0 to 4.0.0 - [Release notes](https://github.com/spatie/laravel-mix-purgecss/releases) - [Changelog](https://github.com/spatie/laravel-mix-purgecss/blob/master/CHANGELOG.md) - [Commits](spatie/laravel-mix-purgecss@3.0.0...4.0.0) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump LICENSE year * New translations validation.php (Dutch) * Link to Exascale sponsor * Extra dependencies were required * Revert "Bump laravel-mix and laravel-mix-purgecss" * Remove Gulp and update requirements lists * Use Twig namespaces * Move overview up * Add documentation * Improve database performance by removing duplicated queries * Typo in phpdoc * Revert "Typo in phpdoc" This reverts commit af94758. * Typo in phpdoc * Bump symfony/dom-crawler from 4.2.1 to 4.2.2 Bumps [symfony/dom-crawler](https://github.com/symfony/dom-crawler) from 4.2.1 to 4.2.2. - [Release notes](https://github.com/symfony/dom-crawler/releases) - [Changelog](https://github.com/symfony/dom-crawler/blob/master/CHANGELOG.md) - [Commits](symfony/dom-crawler@v4.2.1...v4.2.2) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump aws/aws-sdk-php from 3.82.3 to 3.82.6 Bumps [aws/aws-sdk-php](https://github.com/aws/aws-sdk-php) from 3.82.3 to 3.82.6. - [Release notes](https://github.com/aws/aws-sdk-php/releases) - [Changelog](https://github.com/aws/aws-sdk-php/blob/master/CHANGELOG.md) - [Commits](aws/aws-sdk-php@3.82.3...3.82.6) Signed-off-by: dependabot[bot] <support@dependabot.com> * New translations forms.php (Dutch) * New translations dashboard.php (Dutch) * New translations validation.php (Dutch) * New translations cachet.php (Dutch) * Bump fideloper/proxy from 4.0.0 to 4.1.0 Bumps [fideloper/proxy](https://github.com/fideloper/TrustedProxy) from 4.0.0 to 4.1.0. - [Release notes](https://github.com/fideloper/TrustedProxy/releases) - [Commits](fideloper/TrustedProxy@4.0.0...4.1.0) Signed-off-by: dependabot[bot] <support@dependabot.com> * [Security] Bump twig/twig from 2.6.0 to 2.6.2 Bumps [twig/twig](https://github.com/twigphp/Twig) from 2.6.0 to 2.6.2. **This update includes security fixes.** - [Release notes](https://github.com/twigphp/Twig/releases) - [Changelog](https://github.com/twigphp/Twig/blob/2.x/CHANGELOG) - [Commits](twigphp/Twig@v2.6.0...v2.6.2) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump aws/aws-sdk-php from 3.82.6 to 3.86.2 Bumps [aws/aws-sdk-php](https://github.com/aws/aws-sdk-php) from 3.82.6 to 3.86.2. - [Release notes](https://github.com/aws/aws-sdk-php/releases) - [Changelog](https://github.com/aws/aws-sdk-php/blob/master/CHANGELOG.md) - [Commits](aws/aws-sdk-php@3.82.6...3.86.2) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump laravel/framework from 5.7.19 to 5.7.21 Bumps [laravel/framework](https://github.com/laravel/framework) from 5.7.19 to 5.7.21. - [Release notes](https://github.com/laravel/framework/releases) - [Changelog](https://github.com/laravel/framework/blob/5.7/CHANGELOG-5.7.md) - [Commits](laravel/framework@v5.7.19...v5.7.21) Signed-off-by: dependabot[bot] <support@dependabot.com> * New translations cachet.php (German) * New translations dashboard.php (German) * New translations forms.php (German) * New translations notifications.php (German) * Remove duplicate order statement. Fixes cachethq#3418 * Remove string about HTTP(s). Fixes cachethq#3421 * Fixes CORS headers. Closes cachethq#3413 * Apply fixes from StyleCI * Update deps * Compile assets * Fix importing of node_modules CSS * Compile assets * Fully remove the Strengthify integration * Compile assets * New translations cachet.php (Mongolian) * New translations dashboard.php (Mongolian) * New translations forms.php (Mongolian) * New translations pagination.php (Mongolian) * New translations setup.php (Mongolian) * New translations validation.php (Mongolian) * New translations notifications.php (Mongolian) * New translations cachet.php (Slovenian) * New translations dashboard.php (Slovenian) * New translations forms.php (Slovenian) * New translations pagination.php (Slovenian) * New translations setup.php (Slovenian) * New translations validation.php (Slovenian) * New translations notifications.php (Slovenian) * Fix other duplicate order by statements * Apply fixes from StyleCI * Fix HTML encoding for Google Fonts * Update blog feed Fixes cachethq#3441 * Bump sortablejs from 1.7.0 to 1.8.1 Bumps [sortablejs](https://github.com/SortableJS/Sortable) from 1.7.0 to 1.8.1. - [Release notes](https://github.com/SortableJS/Sortable/releases) - [Commits](SortableJS/Sortable@1.7.0...1.8.1) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump vue and vue-template-compiler Bumps [vue](https://github.com/vuejs/vue) and [vue-template-compiler](https://github.com/vuejs/vue). These dependencies needed to be updated together. Updates `vue` from 2.5.21 to 2.5.22 - [Release notes](https://github.com/vuejs/vue/releases) - [Commits](vuejs/vue@v2.5.21...v2.5.22) Updates `vue-template-compiler` from 2.5.21 to 2.5.22 - [Release notes](https://github.com/vuejs/vue/releases) - [Commits](vuejs/vue@v2.5.21...v2.5.22) Signed-off-by: dependabot[bot] <support@dependabot.com> * Do not use arr_* or str_* helpers * Add missing Arr import * Fix setup username validation * Add user on cachet install command * Get a fresh configuration * Use class notation * Apply fixes from StyleCI * Don't prompt for tag files [Fix cachethq#3453] * Substitute :name by incident name on slack title Slack title for created incident has ':name', which is being substituted by incident name. * We no longer need the dev symfony requirements * Bump phpunit/phpunit from 7.5.3 to 7.5.4 Bumps [phpunit/phpunit](https://github.com/sebastianbergmann/phpunit) from 7.5.3 to 7.5.4. - [Release notes](https://github.com/sebastianbergmann/phpunit/releases) - [Changelog](https://github.com/sebastianbergmann/phpunit/blob/master/ChangeLog-7.5.md) - [Commits](sebastianbergmann/phpunit@7.5.3...7.5.4) Signed-off-by: dependabot[bot] <support@dependabot.com> * Change to appropriate tags to hide and select/unselect components in group * Drop dependency on alt-three throttle (cachethq#3469) I've just pulled in the subset of the features we were actually using. * Bump laravel/framework from 5.7.25 to 5.7.26 Bumps [laravel/framework](https://github.com/laravel/framework) from 5.7.25 to 5.7.26. - [Release notes](https://github.com/laravel/framework/releases) - [Changelog](https://github.com/laravel/framework/blob/5.7/CHANGELOG-5.7.md) - [Commits](laravel/framework@v5.7.25...v5.7.26) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump graham-campbell/binput from 6.0.0 to 6.1.0 Bumps [graham-campbell/binput](https://github.com/GrahamCampbell/Laravel-Binput) from 6.0.0 to 6.1.0. - [Release notes](https://github.com/GrahamCampbell/Laravel-Binput/releases) - [Changelog](https://github.com/GrahamCampbell/Laravel-Binput/blob/master/CHANGELOG.md) - [Commits](GrahamCampbell/Laravel-Binput@v6.0.0...v6.1.0) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump graham-campbell/markdown from 10.2.0 to 10.3.0 Bumps [graham-campbell/markdown](https://github.com/GrahamCampbell/Laravel-Markdown) from 10.2.0 to 10.3.0. - [Release notes](https://github.com/GrahamCampbell/Laravel-Markdown/releases) - [Changelog](https://github.com/GrahamCampbell/Laravel-Markdown/blob/master/CHANGELOG.md) - [Commits](GrahamCampbell/Laravel-Markdown@v10.2.0...v10.3.0) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump graham-campbell/testbench-core from 3.0.0 to 3.0.2 Bumps [graham-campbell/testbench-core](https://github.com/GrahamCampbell/Laravel-TestBench-Core) from 3.0.0 to 3.0.2. - [Release notes](https://github.com/GrahamCampbell/Laravel-TestBench-Core/releases) - [Changelog](https://github.com/GrahamCampbell/Laravel-TestBench-Core/blob/master/CHANGELOG.md) - [Commits](GrahamCampbell/Laravel-TestBench-Core@v3.0.0...v3.0.2) Signed-off-by: dependabot[bot] <support@dependabot.com> * Re-enable middleware. Fixes cachethq#3472 * Move meta relation into trait * Remove badly added components class * Improve count call * More UX and UI fixes * Compile assets * [Security] Bump bootstrap-sass from 3.4.0 to 3.4.1 Bumps [bootstrap-sass](https://github.com/twbs/bootstrap-sass) from 3.4.0 to 3.4.1. **This update includes security fixes.** - [Release notes](https://github.com/twbs/bootstrap-sass/releases) - [Changelog](https://github.com/twbs/bootstrap-sass/blob/master/CHANGELOG.md) - [Commits](twbs/bootstrap-sass@v3.4.0...v3.4.1) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump mockery/mockery from 1.2.1 to 1.2.2 Bumps [mockery/mockery](https://github.com/mockery/mockery) from 1.2.1 to 1.2.2. - [Release notes](https://github.com/mockery/mockery/releases) - [Changelog](https://github.com/mockery/mockery/blob/master/CHANGELOG.md) - [Commits](mockery/mockery@1.2.1...1.2.2) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump laravolt/avatar from 2.1.0 to 2.2.0 Bumps [laravolt/avatar](https://github.com/laravolt/avatar) from 2.1.0 to 2.2.0. - [Release notes](https://github.com/laravolt/avatar/releases) - [Changelog](https://github.com/laravolt/avatar/blob/master/CHANGELOG.md) - [Commits](laravolt/avatar@2.1.0...2.2.0) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump phpunit/phpunit from 7.5.4 to 7.5.6 Bumps [phpunit/phpunit](https://github.com/sebastianbergmann/phpunit) from 7.5.4 to 7.5.6. - [Release notes](https://github.com/sebastianbergmann/phpunit/releases) - [Changelog](https://github.com/sebastianbergmann/phpunit/blob/master/ChangeLog-7.5.md) - [Commits](sebastianbergmann/phpunit@7.5.4...7.5.6) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump aws/aws-sdk-php from 3.87.8 to 3.87.17 Bumps [aws/aws-sdk-php](https://github.com/aws/aws-sdk-php) from 3.87.8 to 3.87.17. - [Release notes](https://github.com/aws/aws-sdk-php/releases) - [Changelog](https://github.com/aws/aws-sdk-php/blob/master/CHANGELOG.md) - [Commits](aws/aws-sdk-php@3.87.8...3.87.17) Signed-off-by: dependabot[bot] <support@dependabot.com> * Correctly retrieve the value * Remove duplicate CSS properties Remove unused parameter * Bump barryvdh/laravel-cors from 0.11.2 to 0.11.3 Bumps [barryvdh/laravel-cors](https://github.com/barryvdh/laravel-cors) from 0.11.2 to 0.11.3. - [Release notes](https://github.com/barryvdh/laravel-cors/releases) - [Changelog](https://github.com/barryvdh/laravel-cors/blob/master/changelog.md) - [Commits](fruitcake/laravel-cors@v0.11.2...v0.11.3) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump barryvdh/laravel-debugbar from 3.2.2 to 3.2.3 Bumps [barryvdh/laravel-debugbar](https://github.com/barryvdh/laravel-debugbar) from 3.2.2 to 3.2.3. - [Release notes](https://github.com/barryvdh/laravel-debugbar/releases) - [Changelog](https://github.com/barryvdh/laravel-debugbar/blob/master/changelog.md) - [Commits](barryvdh/laravel-debugbar@v3.2.2...v3.2.3) Signed-off-by: dependabot[bot] <support@dependabot.com> * Make pgsql schema configurable via environment variable * Bump graham-campbell/exceptions from 11.1.0 to 11.2.0 Bumps [graham-campbell/exceptions](https://github.com/GrahamCampbell/Laravel-Exceptions) from 11.1.0 to 11.2.0. - [Release notes](https://github.com/GrahamCampbell/Laravel-Exceptions/releases) - [Changelog](https://github.com/GrahamCampbell/Laravel-Exceptions/blob/master/CHANGELOG.md) - [Commits](GrahamCampbell/Laravel-Exceptions@v11.1.0...v11.2.0) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump aws/aws-sdk-php from 3.87.17 to 3.89.1 Bumps [aws/aws-sdk-php](https://github.com/aws/aws-sdk-php) from 3.87.17 to 3.89.1. - [Release notes](https://github.com/aws/aws-sdk-php/releases) - [Changelog](https://github.com/aws/aws-sdk-php/blob/master/CHANGELOG.md) - [Commits](aws/aws-sdk-php@3.87.17...3.89.1) Signed-off-by: dependabot[bot] <support@dependabot.com> * Bump laravel/framework from 5.7.26 to 5.7.28 Bumps [laravel/framework](https://github.com/laravel/framework) from 5.7.26 to 5.7.28. - [Release notes](https://github.com/laravel/framework/releases) - [Changelog](https://github.com/laravel/framework/blob/5.8/CHANGELOG-5.7.md) - [Commits](laravel/framework@v5.7.26...v5.7.28) Signed-off-by: dependabot[bot] <support@dependabot.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The setup page seems to require a mail host login username (but doesn't require a password). I think the username and password should both be optional.
In my specific use case, I'm setting the mail host to
127.0.0.1
, because my local host already has mail forwarding configured correctly. So I don't need any credentials to send mailThe text was updated successfully, but these errors were encountered: