Skip to content

Commit

Permalink
Merge branch 'main' into release
Browse files Browse the repository at this point in the history
  • Loading branch information
deepansh96 committed Oct 8, 2022
2 parents 5a08e51 + 641e8b9 commit 97a011e
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 0 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -159,3 +159,4 @@ cython_debug/
# and can be added to the global gitignore or merged into this file. For a more nuclear
# option (not recommended) you can uncomment the following to ignore the entire idea folder.
#.idea/
.vscode/settings.json
5 changes: 5 additions & 0 deletions app/routers/reports.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
from fastapi.templating import Jinja2Templates
from fastapi import HTTPException
from collections import OrderedDict
from urllib.parse import unquote

from models.student_quiz_report import StudentQuizReportController

Expand Down Expand Up @@ -46,6 +47,10 @@ def student_quiz_report(request: Request, session_id: str, user_id: str):
status_code=400,
detail="Session ID and User ID have to be specified",
)
# decoding URL encoded values. As this information is coming through a URL,
# it's possible that the strings are URL encoded.
session_id = unquote(session_id)
user_id = unquote(user_id)
try:
data = self.__student_quiz_reports_controller.get_student_quiz_report(
session_id=session_id, user_id=user_id
Expand Down

0 comments on commit 97a011e

Please sign in to comment.