laravel8でメール送信処理を共通化したい

前提

laravelでwebアプリを開発しています

実現したいこと

定期実行している自動ツイート機能が完了したらメールを送れるようにしたい

発生している問題・エラーメッセージ

Trying to get property 'name' of non-object {"exception":"[object] (ErrorException(code: 0): Trying to get property 'name' of non-object

該当のソースコード

App\Console\Commands配下にtwitterへの予約投稿実行機能のクラスを作成し
そこでメール送信用のコントローラーを呼び出しています

php
1    $mail = app()->make('App\Http\Controllers\SendMailController');
2    $mail->sendMailExec();

メール送信用のコントローラーは以下になります

PHP
1<?php
2
3namespace App\Http\Controllers;
4
5use Illuminate\Http\Request;
6use App\Mail\SendMail;
7use Illuminate\Support\Facades\Mail;
8use App\Mail\MailNotify;
9use Illuminate\Support\Facades\Auth;
10
11class SendMailController extends Controller
12{
13    public function index(){
14   　　　 $mail_name = Auth::user()->name;
15             $mail_title = "自動ツイート完了";
16             $mail_text = $mail_name. "様 自動ツイート処理が完了しました。";
17             $mail_to = Auth::user()->email;
18             Mail::to($mail_to)->send( new SendMail($mail_title,$mail_text) );
19
20    }
21}
22

エラーの内容から、メール送信用のコントローラーがlaravelの認証チェックに入っておらず
Authのデータが取れていないことが原因であることまでは推測できています

web.phpへ認証を行うグループ内にルーティングを追加しましたが
エラーは変わっていません

PHP
1Route::group(['middleware' => 'check'],function(){
2    Route::get('/sendMailExec', [App\Http\Controllers\SendMailController::class, 'index'])->name('sendMailExec');
3});

ルーティングの書き方、コントローラの呼び出し方のどちらかが問題だと思うのですが
確認するべき点はどこになるでしょうか？

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

Laravel v8.83.23
PHP v7.4.18

行動規範の内容に同意します

回答1件

ベストアンサー

コントローラーをこんな使い方することは絶対にない。ここから間違ってる。

    $mail = app()->make('App\Http\Controllers\SendMailController');

Auth::user()はブラウザから見た時にしか機能しない。
コマンドを実行した時には「ログインユーザー」なんてどこにもいない。
特定のユーザー情報を使いたいなら

php
1$user = User::find(1);
2$mail_title = "自動ツイート完了";
3$mail_text = $user->name . "様 自動ツイート処理が完了しました。";
4Mail::to($user)->send(new SendMail($mail_title, $mail_text));