Skip to content

Commit

Permalink
login_fix
Browse files Browse the repository at this point in the history
  • Loading branch information
@EugeneLoy
EugeneLoy committed Dec 6, 2019
1 parent 265718c commit 5cb7838
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions edx_dl/edx_dl.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@
}
}
BASE_URL = OPENEDX_SITES['edx']['url']
EDX_HOMEPAGE = BASE_URL + '/login_ajax'
EDX_HOMEPAGE = BASE_URL + '/user_api/v1/account/login_session'
LOGIN_API = BASE_URL + '/login_ajax'
DASHBOARD = BASE_URL + '/dashboard'
COURSEWARE_SEL = OPENEDX_SITES['edx']['courseware-selector']
Expand All @@ -118,7 +118,7 @@ def change_openedx_site(site_name):
sys.exit(ExitCode.UNKNOWN_PLATFORM)

BASE_URL = OPENEDX_SITES[site_name]['url']
EDX_HOMEPAGE = BASE_URL + '/login_ajax'
EDX_HOMEPAGE = BASE_URL + '/user_api/v1/account/login_session'
LOGIN_API = BASE_URL + '/login_ajax'
DASHBOARD = BASE_URL + '/dashboard'
COURSEWARE_SEL = OPENEDX_SITES[site_name]['courseware-selector']
Expand Down Expand Up @@ -1013,12 +1013,12 @@ def main():
# Parse the sections and build the selections dict filtered by sections
if args.platform == 'edx':
all_selections = {selected_course:
get_available_sections(selected_course.url.replace('info', 'course'),
get_available_sections(selected_course.url.replace('info', 'course'),
headers)
for selected_course in selected_courses}
else:
all_selections = {selected_course:
get_available_sections(selected_course.url.replace('info', 'courseware'),
get_available_sections(selected_course.url.replace('info', 'courseware'),
headers)
for selected_course in selected_courses}

Expand Down

2 comments on commit 5cb7838

@abougouffa
Copy link

Choose a reason for hiding this comment

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

This commit kills edx-dl for fun-mooc.fr platform, I modified this to EDX_HOMEPAGE = BASE_URL + '/login_ajax' and it worked for me

@EugeneLoy
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@abougouffa Hi. Yes, you are correct and this is addressed in yet to be merged #574

Please sign in to comment.