How to resolve the cURL error in Laravel 10 SMS sending ?

Level 122

wow, you really have over 700 lines of code in your ApplicantController?

Whatever you did on line 722, failed, and you appear not to catch the exception, letting it bubble through to your ajax request

Since you don't show anything about how you send SMS, noone can help

Level 2

@Snapey My Controller SendOTP functionality is

public function SendOTP(Request $request)
{   
    $mobile = $request->input('mobile_no');

    // Check if an OTP request has been made for this mobile number in the last 20 seconds
    if (!Cache::has('otp_request_' . $mobile)) {
        $pin = mt_rand(100000, 999999);
        $message1 = "KSRTC - " . $pin . " is your OTP for application submission confirmation. Please enter it to proceed.";
        $str = 'username=ksrtcfrontoffic&password=ksrtc123499&message=' . $message1 . "&numbers=" . $mobile . "&senderid=KSRTCMG";
        $url = 'http://api.sms.tn.gov.in/fastclient/SMSclient.php?' . $str;
        session(['otp' => $pin]);
        $response = Http::get($url);

        if ($response->ok()) {
            // Set a rate limit for this mobile number for 20 seconds
            Cache::put('otp_request_' . $mobile, true, 20);

            return response()->json(['success' => true]);
        }
    }

    return response()->json(['success' => false]);
}

Please help

Level 122

So many bad practices

WAIT, you allow the user to specify the mobile number to use? That totally breaks the point of OTP. I thought you had this app security checked?

If you just published your service credentials in this post, edit the question, and request a new password from the provider
put something like this in its own class
extract username, password and url into a service config which gets its values from ENV. Don't bake them into your code. At present If any of the values change you have to redeploy your code. Also, you dont want service credentials stored in your code repo.
catch exceptions thrown by the Http request

The error is that the connection timed out.

Either the url changed, or your credentials are not valid or the service is down

1 like

Level 2

@Snapey and all : That credentials and details are dummy values ( for posting here). My Question is this SMS service and form is working in localhost as shown Localhost working screenshot

It's shown that error in Staging Server . What may be I am wrong ?

Please advise

Level 2

Dear Team, At last I found error reason. There is no active internet connection in staging server. cURL is not working for SMS gateway as

[root@KSRTCApp ~]# curl -I http://api.sms.tn.gov.in
curl: (28) Failed to connect to api.sms.tn.gov.in port 80: Connection timed out

Am I right friends ?

Please advise

Thanks

Anes P A

Level 122

@insight not sure I should bother answering as you clearly know better than to take my advice

If the staging server has no connection to the internet, how did you install the app?

Of you dont have command line access to the server then this must be a toy hobby project, yes?

Level 2

No it's a staging server. I can access server through my network but if server is away from internet it cannot access a resource like sms gateway. Hope you get me ?

Level 122

@insight how did you install the application on the server (ie, composer install etc)

You get me, or perhaps language is a barrier?

Level 2

@Snapey No barrier sir, all installations are over in server. On that time internet availability is there.