Laravel5.7での認可処理

Question

Laravel5.7で認可処理を実装しようとしているのですが、どうしても403になってしまいます。。。

Userクラス：make authで作成し項目を追加。
Visitorクラス：Userクラスが複数持つ事ができる招待者のクラス。

<?php

namespace App\Providers;

use Illuminate\Support\Facades\Gate;
use Illuminate\Foundation\Support\Providers\AuthServiceProvider as ServiceProvider;
use app\Http\Model\Visitor;
use app\Policies\VisitorPolicy;

class AuthServiceProvider extends ServiceProvider
{
    /**
     * The policy mappings for the application.
     *
     * @var array
     */
    protected $policies = [
        //'App\Model' => 'App\Policies\ModelPolicy',
        Visitor::class => VisitorPolicy::class,
    ];

    /**
     * Register any authentication / authorization services.
     *
     * @return void
     */
    public function boot()
    {
        $this->registerPolicies();
    }
}

<?php

namespace App\Policies;

use App\User;
use App\Http\Model\Visitor;
use Illuminate\Auth\Access\HandlesAuthorization;

class VisitorPolicy
{
    use HandlesAuthorization;

    /**
     * Create a new policy instance.
     *
     * @return void
     */
    public function __construct()
    {
        //
    }


    public function view(User $user)
    {
        return true;
    }
}

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Http\Requests\VisitorRequest;
use App\Http\Model\Visitor;
use Illuminate\Support\Facades\Auth;

use App\Exports\VisitorExport;

class VisitorController extends Controller
{

 	/**
     * Create a new controller instance.
     *
     * @return void
     */
    public function __construct()
    {
        $this->middleware(['auth']);
    }



    public function edit(int $id)
    {

        $this->authorize('view', Visitor::class);

    	$data = Visitor::find($id);

        return view('visitor.edit',['data'=>$data]);
    }


    
}

<?php

namespace App;

use Illuminate\Notifications\Notifiable;
use Illuminate\Contracts\Auth\MustVerifyEmail;
use Illuminate\Foundation\Auth\User as Authenticatable;
use App\Notifications\JaPasswordReset;
use App\Http\Model\Chapter;
use App\Http\Model\Visitor;

class User extends Authenticatable
{
    use Notifiable;

    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = [
        'chapter_id','name01','name02','kana01','kana02','company','category','base_comment', 'email', 'password','role'
    ];

    /**
     * The attributes that should be hidden for arrays.
     *
     * @var array
     */
    protected $hidden = [
        'password', 'remember_token',
    ];


    public function visitor()
    {
        return $this->hasMany('App\Model\Visitor' , 'member_id' , 'id');
    }

    public function sendPasswordResetNotification($token)
    {
        $this->notify(new JaPasswordReset($token));
    }
}

<?php

namespace App\Http\Model;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Notifications\Notifiable;

class Visitor extends Model
{

    use Notifiable;


    protected $table = 'visitors';

    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = [
        'visit','company','name01','name02','kana01','kana02','category','member_id','member_name','comment', 'status'
    ];

    /**
     * The attributes that should be hidden for arrays.
     *
     * @var array
     */
    protected $hidden = [
        'password', 'remember_token',
    ];


    public function user()
    {
        return $this->belongsTo('App\User','member_id');
    }


}

どの辺りに問題がありそうでしょうか？
その他、確認が必要なクラスがあれば追記させて頂きます。

Comments

[m.ts10806]

403はLaravelが出力しているものでしょうか？サーバーが出力しているものでしょうか？

[Lab-art_Yamada]

Laravelが出力しています。

Answer 1

どうしたいのか明確に書かれていないので微妙なんですが

一般的には

$this->authorize('view', Visitor::class); // Visitorクラスに関連する操作をできるか？みたいな感じ
$data = Visitor::find($id);

ここが

$data = Visitor::find($id);
$this->authorize('view', $data); // 特定のVisitorインスタンスをログイン中のユーザで表示できるか　みたいな感じ

こうなります。

また、現在は無条件でtrueなので
そのユーザが保有するvisitorのみに限りたい場合は

public function view(User $user, Visitor $visitor)
{
    return $visitor->member_id == $user->id;
}

みたいにする必要があるでしょう

Comments

[Lab-art_Yamada]

ご回答ありがとうございます。
本来ならおっしゃっている様な感じにするのですが、現在の全く無条件の状態でも403になってしまうので困っています。。。

[mikkame]

visitorのインスタンスを入れても403になってしまうでしょうか？

[Lab-art_Yamada]

そうだったんですけど、よく分からないですが解決しました。。。

Answer 2

すみません。解決？しました。

Gitでソースコードを共有している別の環境だと問題なく動作しました。

こういった場合、何が原因なんでしょうかね？

皆さまありがとうございました。
お騒がせしました(^^;