Guide to secure your SmartAPI Account with two factor authentication

[milind017]

@SSumit
I have tried with shared link code, still getting the same

Shell Output:
with TOTP: 932310
With parse_uri: 932310
{'status': False, 'message': 'Invalid totp', 'errorcode': 'AB1050', 'data': None}

[aj_Brk]

@milind017 I see that you have put condition in if data['status'] , is it going in retry at all ?

[aj_Brk]

@milind017 try with this condition

if not data['message'].contains('Invalid totp'):
break

[punit]

@milind017 @aj_Brk why retry, it passed almost always on first time. Anyway. Not sure why parse_uri is needed.. also just pass totp as a string

[gokul]

TypeError: SmartConnect.generateSession() takes 3 positional arguments but 4 were given

[aj_Brk]

@Ashok Actually you are right it works most of the time , but for those rare instances where by the time your request went and it expired on server side , i had added retry. It happened to me once perhaps due to slow code execution or internet.

[aj_Brk]

@gokul try the latest library - pip install smartapi-python --upgrade . It appears the new version is live now

[aj_Brk]

@mudit236 @babitakundu It appears it is updated now. Please try - pip install smartapi-python --upgrade

[milind017]

@Ashok Yes I have already tried with totp as string still its same
Yes retry not needed by still to be sure.
I have added both code to check if I am getting the same totp, which I am getting but still not able to get the success
Is pyotp is working for you?

[admin]

@milind017 @Ashok @gokul @Meenavenkit

Totp changes have been deployed in the latest python release 1.3.0
Please use the following command to upgrade to the latest python release
pip install smartapi-python --upgrade

[milind017]

@aj_Brk Yes retrying working correctly, but I am not able to get the session token
Can you please share reference code

[SSumit]

@milind017 check value of angleOneURI, it should be like - otpauth://totp/<client id>?secret=<secrets>&issuer=angelone.in&algorithm=SHA1&digits=6&period=30

and there is no need for a while loop

[dakshesh]

@milind017 try as int(TOPT)

[milind017]

@SSumit Yes its exactly same except secret key and client id

[milind017]

@dakshesh yes tired same results

[msg2biswajit]

Please update the PHP API for TOTP and share the example for PHP Smart API Because, presently this not working with PHP Smart API

[punit]

@msg2biswajit , please PM me , I will help

[punit]

@milind017 PM me , we can resolve it

[punit]

@milind017 PM me, I will try to help

[msg2biswajit]

@msg2biswajit : I have updated the PHP Smart API function as follows
public static function GenerateSession($clientcode, $password, $totp)

and

$api_parameter = ['clientcode'=>$clientcode,'password'=>$password, 'totp'=>$totp];

now it's working. But, the problem is that I need to update that $totp value manually on every 30 Second, so how can I use that for continuously for whole day ? Please provide the solution