Please add a feature for generating token with some expiry time and in verify api validate the token on the basis of expiry time as well.

i have tried sendOtp.send for sending otp.
And i got the response
{ message: '376941717932393732393030', type: 'success' }
But when i checked it in MSG91 it shows the status as "Rejected by Kannel or Provider "

Hi,

I tried to install and use your package called sendotp : https://www.npmjs.com/package/sendotp

I have created a plugin 'sendotp' in my plugins directory here's the code:

import Vue from 'vue'
import SendOtp from 'sendotp';
const sendOtp = new SendOtp('AuthKey');
Vue.config.productionTip = false
Vue.use(SendOtp, sendOtp)

And added in my plugins list in nuxt.config.js:

plugins: ['~/plugins/sendotp.js']

And getting this error:

TypeError
Class constructor SendOtp cannot be invoked without 'new'

Any Suggestions ?

I will make a PR for this soon, I guess

Hi, I want to know what is the default limit for verify api call?

Also, How can it be set in the dashboard?

I went through the api doc, but I couldn't find any option there.

All I could find this error message,

Max limit reached:

{"message":"Max limit reached for this otp verification","type":"error"}

Link to the doc

To send non english characters(Ex: Arabic), we have to set option unicode = 1

Example:

https://api.msg91.com/api/v5/otp?authkey=<>&extra_param={"unicode":"1"}&mobile=<>&otp=<>

Hi According Node Security You should update your mocha package to avoid few known security issues .

Thank you

This below code works in NodeJS Express framework but when I deployed same code in AWS Lambda function it didn't work.
Here's my demo code.

*** BEGIN ***

const SendOtp = require('sendotp');
const objSendOtp = new SendOtp('<API_SECRET>');

objSendOtp.send(, , , function(err, data){
if(err) console.log(err);
else
console.log(data);
});

*** END ***

Hi,
Is there any feature of verifying the otp only once and then the otp expires after one successful verification.

Incase, the feature is present, kindly let me know how to use it, if not then will that be added soon ?

I have used otp expiry, but in my implementation, I need it to expire after one successful verification.

The current mechanism of working with OTP DOEN NOT have a mechanism to use Promises. That might be needed, as working with a combination of async/await makes it diffucult to use the api

Message template being passed is not being picked, support says that DLT_TE_ID flag needs to be passed as param in request

const Otp = new sendotp('AUTH_KEY','templateid'); can i set template id like this its not working how can we set template id and send the otp

exports.handler = async (event, context, callback) => {
const SendOtp = require('sendotp');
const sendOTP = new SendOtp('260981AZN8sH3O5c551802');
sendOTP.send(mobileNum,"611332", (error, data) => {
if(err) {
return err;
} else {
const response = {
statusCode: 200,
body: data.message,
};
return response;
}
})

this is my lambda function and when i invoke the api endpoint i get null as result and nothing more. The function block of sendOTP is not working.

Using below code to send message
sendOtp.send(req.mobileNumber, 2424, (error, data) => {
console.log("%%%%", data);
}); //otp is optional if not sent it'll be generated automatically

Getting error:
POST /api/auth/otp - - ms - -
%% { message: 'Some Parameter are missing : mobiles/group_id',
type: 'error' }

setting expiry time for otp is not supported yet. can you please provide that.

Please generate @types/sendotp module or .d.ts, the auto-generated index file is not working.

AUTO-GENERATED INDEX

export = index;
declare class index {
static doRequest(method: any, path: any, params: any, callback: any): void;
static generateOtp(): any;
static getBaseURL(): any;
constructor(authKey: any, messageTemplate: any);
authKey: any;
messageTemplate: any;
otp_expiry: any;
retry(contactNumber: any, retryVoice: any, callback: any): any;
send(contactNumber: any, senderId: any, otp: any, callback: any): any;
setOtpExpiry(otp_expiry: any): void;
verify(contactNumber: any, otp: any, callback: any): any;
}

The README has some punctuation problems while showing function definitions and also while showing function calls - like for verify() function call

Hi , i'm unable to send OTP globally to different countries. I have talked to msg91 support and they are saying to add a field country=0. https://help.msg91.com/article/76-how-to-send-sms-worldwide

My question is where to add that field

Trying to send international OTP, doesn't work.
It will require an extra parameter to be sent "country=0" to the API for it to be considered as international OTP SMS.

if I use retryVoice = true , it does not work , fail with "Uncaught exception" however working perfectly with false

sendOtp.retry(phone, false, function (error, data, res) {
if(data.type == 'success'){
console.log('resent successfully');
admin.database().ref('/users/' + phone)
.update({ code: code, codeValid: true }, () => {
response.send({ success: true });
});
}
else {
return response.status(422).send(err);
}
});

The way by which promises has been implemented prevents the callbacks to be called. The logic of allow_promise also incorrect as its always undefined. Will try to post a fix.

Hi,

currently I'm developing React Native where i need to use MSG91 for Use register OTP Verification. but I'm getting Current error while compiling

bundling failed: Error: Unable to resolve module cryptofrom/Users/apple/Desktop/Projects/RN/REG/node_modules/request/lib/helpers.js: Module crypto does not exist in the Haste module map

May be this npm package wont work in react native. please help me asap.

Error:-
request is not defined
at Function.doRequest (/node_modules/sendotp/index.js:161:9)

Code in index.js:-
request(options, function(error, response, data) {
// request error
if (error) {
return callback(error, data, response);
}

        // JSON parse error or empty strings
        try {
            // An empty string is a valid response
            if (data === '') {
                data = {};
            }
            else {
                data = JSON.parse(data);
            }
        }
        catch(parseError) {
            return callback(
                new Error('JSON parseError with HTTP Status: ' + response.statusCode + ' ' + response.statusMessage),
                data,
                response
            );
        }


        // response object errors
        // This should return an error object not an array of errors
        if (data.errors !== undefined) {
            return callback(data.errors, data, response);
        }

        // status code errors
        if(response.statusCode < 200 || response.statusCode > 299) {
            return callback(
                new Error('HTTP Error: ' + response.statusCode + ' ' + response.statusMessage),
                data,
                response
            );
        }
        // no errors
        callback(null, data, response);
    });

Another example code issue with README. Callback function has buggy code. "data" object has key "type" now, and not "msgType"

These build tools are unnecessarily getting added into our build process and giving deprecation warning. Please move them into devDependencies in package.json

How to use promise with send function?
In simple words, I don't want callback in send function.

If you could provide checkStatus method that was available previously on this sdk , that would be very helpful .