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.

Sign up for GitHub

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

Jump to bottom

class names that end with '/' lead to misleading error #155

Open

hoell opened this issue Nov 2, 2014 · 1 comment

hoell commented Nov 2, 2014

Really small issue - perhaps do some sanity checks on the class name..

Run script with a classname that ends with a / character
coursera-dl -u uuu -p ppp class-001/

Observed: Error message "root[main] Could not authenticate: Did not recieve csrf_token cookie."

Expected: Something like "incorrect class name format" or at least "class not found"

Note: this only affected my as I created a download-all.sh script to update all classes in a certain sub folder

Owner

dgorissen commented Nov 2, 2014

Pull request? :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment