CanadianBacon/CanvasRefined
1
0
Fork 0
mirror of https://github.com/GuySandler/CanvasRefined.git synced 2026-11-03 13:57:17 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity

slight fix to canvas auto detect

Browse Source
This commit is contained in:
Guy Sandler 2026-10-08 17:31:43 -07:00
parent 247007defb
commit 1b29e80f19
1 changed files with 15 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
js/content.js
View File

@ -803,7 +803,7 @@ function startReminderMode() {
async function detectCanvasPage() {
try {
const courses = await getData(`${domain}/api/v1/courses?${/*enrollment_state=active&*/""}per_page=100`);
if (Array.isArray(courses) && courses.length) {
if (isCanvasCourseList(courses)) {
return { isCanvas: true, courses };
}
console.log("Canvas Refined - this url doesn't seem to be a canvas url (1)");
@ -814,6 +814,20 @@ async function detectCanvasPage() {
}
}
// Verify the /api/v1/courses response actually looks like Canvas course
// objects, not just "some JSON array". Real Canvas entries always carry a
// numeric id, a name, and identifying fields (uuid/course_code/enrollments).
// Without this check unrelated sites that coincidentally serve their own
// JSON at the same path were misdetected as Canvas — e.g.
// apstudents.collegeboard.org, whose AP-course list uses the shape
// { course_id, course_name, course_url }.
function isCanvasCourseList(courses) {
return Array.isArray(courses) && courses.length > 0 && courses.every(c =>
c && c.id != null && typeof c.name === "string" &&
(c.uuid != null || c.course_code != null || Array.isArray(c.enrollments))
);
}
// Register the domain in the canvas list. `courses` (from detectCanvasPage) is
// passed to getCards so this school's courses are indexed before the reload,
// exactly like the original first-run flow did.