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

Fix 500 internal server error due to lms_user_id not found #61

Conversation

Faraz32123
Copy link
Collaborator

No description provided.

@Faraz32123 Faraz32123 self-assigned this Dec 11, 2023
Close #59.
This error was occuring due to lms_user_id not_found.
Requests that are sent to Ecommerce are expected to include the
LMS user id. If they do not include the id, the creator of the request
is responsible for ensuring that Ecommerce already knows the
LMS user id for the user. Requests to Ecommerce will be rejected and
a MissingLmsUserIdException will be raised if Ecommerce cannot find a
LMS user id for the affected user(s), unless the
allow_missing_lms_user_id waffle switch is enabled.
@Faraz32123 Faraz32123 force-pushed the fix/faraz-fix_500_internal_server_error_due_to_lms_user_id_not_found branch from 90b04fc to f93b2da Compare December 11, 2023 11:18
Copy link
Contributor

@regisb regisb left a comment

Choose a reason for hiding this comment

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

Nice!

@regisb regisb merged commit 4e7845e into master Dec 11, 2023
1 check passed
@regisb regisb deleted the fix/faraz-fix_500_internal_server_error_due_to_lms_user_id_not_found branch December 11, 2023 11:22
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

Successfully merging this pull request may close these issues.

2 participants