Skip to content

Better user errors - #1

Open
leftrightleft wants to merge 1 commit into
dsp-testing:mainfrom
advanced-security-demo:new-feature
Open

Better user errors#1
leftrightleft wants to merge 1 commit into
dsp-testing:mainfrom
advanced-security-demo:new-feature

Conversation

@leftrightleft

Copy link
Copy Markdown

Introducing a new feature!

Comment thread server/routes.py
return make_response(f"No search results for Name: {name}", 404)

elif author:
cursor.execute(

Check warning

Code scanning / CodeQL

Reflected server-side cross-site scripting

Cross-site scripting vulnerability due to [a user-provided value](1).
Comment thread server/routes.py
return make_response(f"No search results for Author: {author}", 404)

else:
cursor.execute("SELECT name, author, read FROM books")

Check warning

Code scanning / CodeQL

Reflected server-side cross-site scripting

Cross-site scripting vulnerability due to [a user-provided value](1). Cross-site scripting vulnerability due to [a user-provided value](2).
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.

3 participants