Laravel Türkiye Discord Kanalı Forumda kod paylaşılırken dikkat edilmesi gerekenler!Birlikte proje geliştirmek ister misiniz?

Api'den gelen İstekleri Yanlış bir şey Olursa Nasıl Sonuç Dönebilirim ?

koti42

Mesela api linki yanlış girildi 404 dönmek istiyorum yada eksik bir bilgi gönderildi validation'a takıldı bu validete mesajlarını nasıl gösterebilirim örnek çoğaltılabilir sanırım. Bu konu da yardımcı olabilir misiniz.

try catch ile mi yapmam gerekiyor yoksa bunları nasıl yakalayabilirim.

https://prnt.sc/Ei97lcgZLcfn bu mesaj dönüşü doğru mu bir sürü soru işareti oluştu kafam da

koti42

Bir de ek olarak bu route::fallback komutum çalışmıyor linki hataları da girsem neden olabilir acaba ?

<?php

Route::group([
  'prefix' => 'roket',
  'as' => 'roket.'
], function(){
    Route::post('payment', 'RocketPaymentController@payment')->middleware('auth.apikey');
    Route::get('products/', 'RocketPaymentController@getProductListAll');
    Route::get('products/online_education', 'RocketPaymentController@getProductListAllOnline');
    Route::fallback(function(){
        return response()->json([
            'message' => 'Page Not Found. If error persists, contact info@edanisman.com'], 404);
    });

});

koti42

Bu Apide ki geri dönüşleri özelleştirmek Laravel 5.5 kullanıyor ve kaynak bulamıyorsanız Ölüm 😃
@mgsmus Abi burada ki validation mesajlarından birisi bile sağlanmasa hepsini basıyor ekrana bunları doğru düzgün yapamaz mıyım bir yol gösterir misin 😃


        try {
            $user=User::whereEmail($request->email)->first();
            $registerData = $this->validate($request, [
                'first_name' => 'required',
                'last_name' => 'required',
                'phone_number' => 'required|min:10',
                'email' => 'required',
                'company_name' => 'required',
                'company_type' => 'required',
                'tax_number' => 'required',
                'tax_office' => 'required',
                'invoice_city' => 'required',
                'invoice_district' => 'required',
                'grand_total' => 'required',
                'parampos_id' => 'required',
                'product.*.product_id' => 'required',
                'product.*.product_price_id' => 'required',
                'product.*.qty' => 'required',
                'product.*.sub_total' => 'required',
                'product.*.tax' => 'required',
                'product.*.tax_total' => 'required',
                'product.*.currency' => 'required',
                'product.*.grand_total' => 'required',

                // 'info_form'    => 'required|accepted',
            ],
                [
                    'first_name.required' => 'Adınızı girmeniz gerekmektedir.',
                    'last_name.required' => 'Soyadınızı girmeniz gerekmektedir.',
                    'phone_number.required' => 'Telefon numaranızı girmeniz gerekmektedir.',
                    'phone_number.min' => 'Telefon numaranızı doğru girmeniz gerekmektedir.',
                    'phone_number.unique' => 'Telefon numarası kullanılmaktadır.',
                    'email.min' => 'E-Mail adresinizi girmeniz gerekmektedir.',
                ]);

            if(is_null($user))
            {
                $registerData['password'] = \Hash::make('c{SbG9Tw%ne~\?Ss');
                if ($registerData['company_type'] == 'individual') {
                    $companyType = 0;
                } else {
                    $companyType = 1;

                }
                $registerData['confirmation_code'] = md5(uniqid(mt_rand(), true));
                $registerData['expired_at'] = Carbon::now()->addDays(21);
                $registerData['confirmed'] = 1;
                $registerData['partner_token_id'] = 3;
                $user = User::create($registerData);
                $user->assignRole('business');
                $addresses = $user->addresses()->create([
                    'first_name' => $registerData['first_name'],
                    'last_name' => $registerData['last_name'],
                    'phone_number' => $registerData['phone_number'],
                    'user_id' => $user->id,
                    'company_name' => $registerData['company_name'],
                    'company_type' => $companyType,
                    'is_company' => $companyType,
                    'tax_number' => $registerData['tax_number'],
                    'tax_office' => $registerData['tax_office'],
                    'invoice_city' => $registerData['invoice_city'],
                    'invoice_district' => $registerData['invoice_district'],
                ]);
            }

            $addressId = $addresses->id ?? $user->addresses()->first()->id;

            $payment = $user->payments()->create([
                'price' => $registerData['grand_total'],
                'address_id' => $addressId,
                'name_lastname' => $user->full_name,
                'zipcode' => '34000',
                'payment_status' => 1,
                'day' => 0,
                'type' => 'year',
                'mdStatus' => 1,
                'source' => 'Roket',
                'payment_type'=>'credit_card'
            ]);
            $payment->logs()->create([
                'data' => $registerData['parampos_id'],
                'user_id' => $user->id,
            ]);
            if (is_array($request->input('product'))) {
                foreach ($registerData['product'] as $product) {
                    $product_data = Product::find($product['product_id']);
                    $item_data = ProductPrice::find($product['product_price_id']);
                    $payment->data()->create([
                        'product_price_id' => $product['product_price_id'],
                        'product_id' => $product['product_id'],
                        'sub_total' => $product['sub_total'],
                        'product_data' => $product_data,
                        'item_data' => $item_data,
                        'tax' => $product['tax'],
                        'qty' => $product['qty'],
                        'tax_total' => $product['tax_total'],
                        'grand_total' => $product['grand_total'],
                        'payment_status' => PaymentStatus::SUCCESS,
                        'currency' => $product['currency'],
                        'user_id' => $user->id,
                    ]);
                }
            }

            return response()->json([
                'message' => 'Payment has beed added succesfully',
                'data' => [
                    'user_id' => $user->id,
                ],
                'status' => 'success',
            ], 200);
        }
        catch (\Exception $exception)
        {
            return response()->json([
                'message' => $exception,
                'data' => [
                ],
                'status' => 'error',
            ], 422);
        }

coder2

Validation hatasının obje olarak ekrana basıldığı request headers te "Accept:application/json" gönderiyor musunuz ?

Route::fallback tanımlamasını global scope alarak deneyebilir misin

koti42

coder2 bu şekilde yakalamaya çalıştım fakat yine yakalayamadım istek atıyorum 200 e düşüyor yine

public function render($request, Exception $exception)
{
    if ($exception instanceof NotFoundHttpException) {
        return response()->view('errors.404', [], 404);
    }

    return parent::render($request, $exception);
}

mkeremcansev

koti42

        $this->renderable(function (NotFoundHttpException $exception) {
            //işlem
        });

        return parent::render($request, $e);

koti42

mkeremcansev render fonksiyonun içinde dd atıyorum acaba ekrana basar mı diye sanki oraya hiç girmiyor gibi. Bununla ilgili bir log kaydı oluşuyor mu acaba

mkeremcansev

koti42 paylaştığım kod 404 durum kodu için. Eğer başka bir exception dönüyorsa onu kontrol etmelisin.

koti42

mkeremcansev Bu proje de bununla alakalı bir problem var 404'ü de yakalayamıyorum 😃

namesiz

koti42
api ile haberleşmenizin json ile gerçekleştirildiğini varsayarak render fonksiyonu içerisinde $request->expectsJson() ekleyerek deneyebilir misiniz?

if ($exception instanceof NotFoundHttpException && $request->expectsJson()) {
            return response()->json([
                'status' => 0,
                'message' => __('404 mesajınız hard coded yada dile bağımlı'),
                'data' => []
            ], 404);
        }

koti42

namesiz Yine yakalamadı çok tuhafıma gidiyor artık eski projelerde çalışmak.

Render için de bu kısım yakalıyor

        if ($exception instanceof UnauthorizedException) {
            return redirect()
                ->route(home_route())
                ->withFlashDanger(__('auth.general_error'));
        }

ama json kısımlarını yakalayamıyorum

mgsmus

koti42 Bir tane middleware oluşturun ve app/Http/Kernel.php içindeki $middlewareGroups dizisindeki api anahtarının ilk elemanı olarak ekleyin. Middleware içeriği:

public function handle(Request $request, Closure $next)
{
    $request->headers
        ->set('Accept', 'application/json');

    return $next($request);
}

Böylece API isteklerinin hepsinin json beklediğini istemci göndermese bile kendi tarafınızda kesinleştirmiş olursunuz, namesiz bahsettiği $request->expectsJson() her zaman çalışacaktır.

namesiz

koti42
öncelikle yoğun tempom nedeniyle geç cevaplayabildiğim için ve laravel versiyonunuza bakmadan cevap vererek neden olduğum zaman kaybı için üzgünüm.

her versiyon için dokümantasyon laravel'in kendi sitesinde mevcut, ilk olarak ordan başlamak gerekiyor. Size gerekli olan versiyon için link şöyle ve ajax çağrılarında tüm hatayı 422 response kodu ile dönerim der;
https://laravel.com/docs/5.5/validation#quick-ajax-requests-and-validation

normal çağrılarda ilk attribute validasyonunda hata almasını istiyorsanız bail eklemeniz gerekir;

$request->validate([
    'title' => 'bail|required|unique:posts|max:255',
    'body' => 'required',
]);

ilk mesajınızdaki ekran görüntüsüne baktığımda api/v1 şeklinde link gördüğümden ve laravel versiyonunuza dikkat etmeden $request->expectsJson() şeklindeki kod eklemeyi önermiştim. 🙁

Sonuç olarak ben olsam ne yapardım dediğimde;

api tarafı için bir tane base controller hazırlar, gelen request'in json olup olmadığını teyit eder, custom hata throw ederdim. success ve error response için function hazırlardım. Diğer tüm api controller'ları bu base'den türeterek response'ları standart ve objective hale getirirdim.
Not: Son mesajınızdaki render içinde UnauthorizedException yakalamanız bambaşka kapıları açıyor 😉

koti42

mgsmus Hepinize çok teşekkür ederim. Ama Sonuç doğru ise dd ile bir veri yakalayabiliyorum. girdiğim api linki yok ise middleware uğramıyor veya Handler da yakalayamıyorum bu kısmı.

Eklediğim şekil şöyle.

Middleware Dosyası

<?php

namespace App\Http\Middleware;

use Closure;

class ApiNotFoundJson
{
    /**
     * Handle an incoming request.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  \Closure  $next
     * @return mixed
     */
    public function handle($request, Closure $next)
    {
        $request->headers
            ->set('Accept', 'application/json');

        return $next($request);
    }
}

Kernel Dosyası

    protected $middlewareGroups = [
        'web' => [
            \App\Http\Middleware\EncryptCookies::class,
            \Illuminate\Cookie\Middleware\AddQueuedCookiesToResponse::class,
            \Illuminate\Session\Middleware\StartSession::class,
            // \Illuminate\Session\Middleware\AuthenticateSession::class,
            \Illuminate\View\Middleware\ShareErrorsFromSession::class,
            \App\Http\Middleware\VerifyCsrfToken::class,
            \Illuminate\Routing\Middleware\SubstituteBindings::class,
            \App\Http\Middleware\LocaleMiddleware::class,
            // \App\Http\Middleware\SupplierMiddleware::class
        ],

        'api' => [
            ApiNotFoundJson::class,
            'throttle:500,1',
            'bindings',
            // \App\Http\Middleware\AccountType::class
        ],

        'admin' => [
            'auth',
            'password_expires',
            'permission:view backend',
        ],

        'sessions' => [
            \Illuminate\Session\Middleware\StartSession::class,
        ]
    ];

Handler Dosyası

  public function render($request, Exception $exception)
    {
        if ($exception instanceof NotFoundHttpException && $request->expectsJson()) {
            return response()->json([
                'status' => 0,
                'message' => __('404 mesajınız hard coded yada dile bağımlı'),
                'data' => []
            ], 404);
        }
        if ($exception instanceof UnauthorizedException) {
            return redirect()
                ->route(home_route())
                ->withFlashDanger(__('auth.general_error'));
        }

        return parent::render($request, $exception);
    }

namesiz

koti42 çözüm bulabildin mi?
kodun tamamını görmemekle birlikte gerçekten garip bir durum.
nedenin gözden kaçan küçük bir şey olduğuna inanmaya başladım.
çözümü bulduysan nedenini/çözümünü burada paylaşmak, sonradan ziyaret edenler ve okuyanlar için yol gösterici olacaktır.

koti42

namesiz Malesef. Laravel sürümü eski olduğu ve benden önce kodlama yapıldığı için tespit edemedim