Andal.LND/frontend-app-learning
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: stop logging course-blocks 403 responses as errors (#637)

Browse Source 
They are benign and normal for logged out users. Instead, log them
as info messages, so we can still track them if we need to.

AA-1011
This commit is contained in:
Michael Terry
2021-09-13 15:31:43 -04:00
committed by GitHub
parent 3da1fb6581
commit 54d96cc162
1 changed files with 9 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/courseware/data/thunks.js
View File

@@ -1,4 +1,4 @@
import { logError } from '@edx/frontend-platform/logging';
import { logError, logInfo } from '@edx/frontend-platform/logging';
import {
getBlockCompletion,
postSequencePosition,
@@ -188,7 +188,14 @@ export function fetchCourse(courseId) {
// Log errors for each request if needed. Course block failures may occur
// even if the course metadata request is successful
if (!fetchedBlocks) {
logError(courseBlocksResult.reason);
const { response } = courseBlocksResult.reason;
if (response && response.status === 403) {
// 403 responses are normal - they happen when the learner is logged out.
// We'll redirect them in a moment to the outline tab by calling fetchCourseDenied() below.
logInfo(courseBlocksResult.reason);
} else {
logError(courseBlocksResult.reason);
}
}
if (!fetchedMetadata) {
logError(courseMetadataResult.reason);