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

Setting default account creates errors #13

Closed
ghost opened this issue Mar 17, 2018 · 2 comments
Closed

Setting default account creates errors #13

ghost opened this issue Mar 17, 2018 · 2 comments
Labels
Improve Request to improve a function or component
Milestone

Comments

@ghost
Copy link

ghost commented Mar 17, 2018

./redsea.py list

ERROR: No default session has been set!

SESSIONS:
  au
  us2
  fr
  us

Please provide the name of the session you would like to set as default: au
Default session has successfully been set to "au"
Traceback (most recent call last):
  File "./redsea.py", line 147, in <module>
    main()
  File "./redsea.py", line 72, in main
    media_to_download = cli.parse_media_option(args.urls)
  File "/mnt/c/Users/bamtan/Documents/redsea/redsea/cli.py", line 64, in parse_media_option
    ci = m.index(':')
ValueError: substring not found

after setting default and running readsea.py list

./redsea.py list

Traceback (most recent call last):
  File "./redsea.py", line 147, in <module>
    main()
  File "./redsea.py", line 72, in main
    media_to_download = cli.parse_media_option(args.urls)
  File "/mnt/c/Users/bamtan/Documents/redsea/redsea/cli.py", line 64, in parse_media_option
    ci = m.index(':')
ValueError: substring not found
@redsudo
Copy link
Owner

redsudo commented Mar 19, 2018

So this is actually a syntax error. Command should be redsea.py auth list (you are missing 'auth').

What is occurring is that redsea is loading, asking for a default account, then proceeding and trying to process 'list' as if it were a url instead of a command. It then errors out.

That should get you functional again. In the meantime, let's leave this open as a request for improved error handling of non-url strings.

@redsudo redsudo added the Improve Request to improve a function or component label Mar 19, 2018
@redsudo redsudo added this to the 1.2 milestone Mar 19, 2018
@redsudo redsudo modified the milestones: 1.2, 1.1 Apr 24, 2018
@redsudo
Copy link
Owner

redsudo commented Apr 24, 2018

I have improved URL validation in 1.1 to properly identify and report this error to the user

@redsudo redsudo closed this as completed Apr 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Improve Request to improve a function or component
Projects
None yet
Development

No branches or pull requests

1 participant