Profilsuche nach Nickname #41

Merged
weeman merged 1 commits from feature-nickname-search into main 2021-07-26 19:40:34 +02:00
Owner
  • Suche nach Nickname
  • Den Test mal richtig gemacht - lol

closes #37

- Suche nach Nickname - Den Test mal richtig gemacht - lol closes #37
weeman added 1 commit 2021-07-12 21:15:35 +02:00
add search by nick
All checks were successful
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is passing
f2ea02f7e3
frlan reviewed 2021-07-26 18:26:59 +02:00
@ -22,0 +22,4 @@
if "nickname" in request.args:
nickname = request.args.get("nickname")
query = query.filter(Profile.nickname.like(f"%{nickname}%"))
Owner

Possible SQL injection? Needs to be double checked

Possible SQL injection? Needs to be double checked
Author
Owner

Laut Doku escaped SQLAlchemy das automagisch.
Aber lass einen Test reinpacken.

Laut Doku escaped SQLAlchemy das automagisch. Aber lass einen Test reinpacken.
Author
Owner

Test eingefügt

Test eingefügt
weeman marked this conversation as resolved
weeman added 1 commit 2021-07-26 19:30:58 +02:00
add sql special chars test
All checks were successful
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is passing
840cf8830d
weeman force-pushed feature-nickname-search from 840cf8830d to 76e95311f0 2021-07-26 19:31:36 +02:00 Compare
weeman merged commit ea57f326b7 into main 2021-07-26 19:40:33 +02:00
weeman deleted branch feature-nickname-search 2021-07-26 19:40:38 +02:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: kompetenzinventar/ki-backend#41
No description provided.