Skip to content
This repository has been archived by the owner on Feb 8, 2024. It is now read-only.

Login procedure change and bugs fixing #3

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

Vikassm73
Copy link

New version contain changes in login procedure as well as done some bugs fixing

@manzoor22233
Copy link

manzoor22233 commented Oct 9, 2022

@Vikassm73 API ERROR AND INVALIDHEADER

Updation in Guest Login
@Vikassm73
Copy link
Author

Correction done in new updates

@vikasvisking
Copy link

vikasvisking commented Oct 10, 2022

good work @Vikassm73 @manzoor22233
@iMro0t @nikhilk0 Please review and merge ,
As its not working in App

@manzoor22233
Copy link

@Vikassm73 same error

@Vikassm73
Copy link
Author

Just do logout once and try again

Correction in api
@manzoor22233
Copy link

manzoor22233 commented Oct 11, 2022

@Vikassm73 i could only watch movie as guest....while trying for login it is showing as subscription error or 400: invalid response from sever...anyway thanksss

Correction in Login
@Vikassm73
Copy link
Author

Please try now

@manzoor22233
Copy link

manzoor22233 commented Oct 13, 2022

@Vikassm73 i can logged in successfully..but if i want to watch movies or shows,i have the only "search" option. while clicking on desktop option it is showing as 404: invalid response from server.....thanks for your hard works AND otherwise all are perfect working

Changes in Menuitem
@Vikassm73
Copy link
Author

Vikassm73 commented Oct 14, 2022

Till yesterday, menus were working fine. Now updated new changes

@manzoor22233
Copy link

@Vikassm73 wow perfect job.....thank u so much

Copy link
Member

@iMro0t iMro0t left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove __pycache__ and made above changes so i can merge it

addon.xml Outdated
<requires>
<import addon="xbmc.python" version="3.0.0"/>
<import addon="script.module.codequick" version="1.0.2"/>
<import addon="script.module.inputstreamhelper" version="0.5.2"/>
<import addon="repository.botallen" version="2.0.0"/>
<import addon="script.module.web-pdb"/>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is use of this dependency?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not required, just for debug purpose. I will remove it

url, headers={"x-country-code": "in", "x-platform-code": "ANDROID_TV"})
return deep_get(resp, "body.results.menuItems")
url, headers={"x-country-code": "in", "x-platform-code": "PCTV"})
return deep_get(resp["body"]["results"]["menuItems"][0]["subItem"][1], "subItem")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this can lead to runtime error.

Copy link
Author

@Vikassm73 Vikassm73 Oct 18, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reverse back to old code

@@ -69,16 +69,7 @@ def play_ext(plugin, contentId, partner=None):

@Script.register
def login(plugin):
msg = "1. Go to [B]https://tv.hotstar.com[/B]\n2. Login with your hotstar account[CR]3. Enter the 4 digit code : "
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this dialog is necessary for login.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Previous method of login using login code is no longer working. That's why I have removed this. Now, I have used OTP based method.

@@ -31,19 +35,39 @@ def login_wrapper(*args, **kwargs):
else:
# login require
Script.notify(
"Login Error", "Please login to watch this content")
executebuiltin(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is good way to directly provide login popup when content is not subscribed so user will not be confuse

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I have not subscribed then popup of login screen may be annoying. Just simple message could solve the purpose.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you pls provide the updated addon link for Hotstar 1.2.1?

@k4976
Copy link

k4976 commented Nov 7, 2022

still login error and why i need login to watch free content that i can also watch it on hotstar for absolutely free of cost
when i play video i got error 403 client error : forbidden for usl : https://api.hotstar.com/in/aadhar/v2/firetv/in/user/guest-signup

@Vikassm73
Copy link
Author

Vikassm73 commented Nov 7, 2022

No need to login if you are using it for Free content. Just log out once and try to play any free content without login. Try couple of time if it just failed without any error. Your message shows that you have not downloaded and install from my repository

@narasimharao-nerusu
Copy link

New version contain changes in login procedure as well as done some bugs fixing

@Vikassm73 I am new to this community and want to know how to get the new version of this Hotstar 1.2.1 on my firetv stick please?

@Vikassm73
Copy link
Author

Follow link
Vikassm73/AjaykRepo#109

Do further communication there only.

@79061
Copy link

79061 commented Nov 22, 2022

Please help me I uses https://kido.botallen.com by addon this i get jiotv and hotstar but I'm unable to login into hotstar please help me

Correction in login
Correction in login
@appujet
Copy link

appujet commented Feb 5, 2024

Getting this issue

Uploading VID20240205134431.mp4…

@appujet
Copy link

appujet commented Feb 5, 2024

Uploading Screenshot_2024-02-05-13-49-32-43_99c04817c0de5652397fc8b56c3b3817.jpg…

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants