Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Version Number #144

Closed
Taupe opened this issue May 8, 2024 · 3 comments
Closed

Version Number #144

Taupe opened this issue May 8, 2024 · 3 comments

Comments

@Taupe
Copy link

Taupe commented May 8, 2024

Hey,

I'm getting this pop-up every time I try to start NSO-RPC telling me to enter the App Store version of the NSO Mobile App. After entering the correct version (2.10.0 as of May 8, 2024), nothing happens.

image

@Jiralite
Copy link

Jiralite commented May 8, 2024

The reason why nothing happens is because of this:

def __init__(self, session_token, user_lang, targetID, version):
pattern = re.compile(r'(\d.\d.\d)')
if not version or not re.search(pattern, version):
raise Exception('missing app version')

2.10.0 does not match \d.\d.\d due to it only accepting a singular digit (. itself not being escaped is a separate issue).

The actual cause of this issue is actually from the same check:

def getVersion():
for i in range(5):
try:
r = requests.get('https://apps.apple.com/us/app/nintendo-switch-online/id1234806557', timeout = 10)
break
except requests.RequestException as e:
log(f'Failed to get Apple\'s store page. Retrying... Error: {str(e)}')
else:
log('Failed to get Apple\'s store page after multiple retries.')
if r:
searchPattern = re.compile(r'Version\s*(\d\.\d\.\d)+')
version = searchPattern.search(r.text)
if version:
return version.group(1)
return ''

@HotaruBlaze
Copy link
Contributor

I'm already aware of this and dealt with it here: #145

either do 2.9.1 and hope it wont crash, or use this build for now: https://github.com/HotaruBlaze/NSO-RPC/releases/tag/dev-ovr-2.9.1-HotaruBlaze

@Taupe
Copy link
Author

Taupe commented May 9, 2024

Thanks!

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

No branches or pull requests

3 participants