Nulo/gitea
Archived
1
0
Fork 0
Code Issues Pull requests Projects Releases Activity

Quote references to the user table in consistency checks (#18072) (#18073)

Browse source 
Backport #18072

Although #17487 ensured that the table was quoted in the join it missed that the
query part of the check also needed to be quoted.

Fix #17485

Signed-off-by: Andrew Thornton <art27@cantab.net>
This commit is contained in:
zeripath 2021-12-22 23:29:05 +00:00 committed by GitHub
parent 6100935a77
commit 6eaebda1b5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
modules/doctor/dbconsistency.go
View file

@ -159,13 +159,13 @@ func checkDBConsistency(logger log.Logger, autofix bool) error {
"lfs_lock", "repository", "lfs_lock.repo_id=repository.id"), "lfs_lock", "repository", "lfs_lock.repo_id=repository.id"),
// find collaborations without users // find collaborations without users
genericOrphanCheck("Collaborations without existing user", genericOrphanCheck("Collaborations without existing user",
"collaboration", "user", "collaboration.user_id=user.id"), "collaboration", "user", "collaboration.user_id=`user`.id"),
// find collaborations without repository // find collaborations without repository
genericOrphanCheck("Collaborations without existing repository", genericOrphanCheck("Collaborations without existing repository",
"collaboration", "repository", "collaboration.repo_id=repository.id"), "collaboration", "repository", "collaboration.repo_id=repository.id"),
// find access without users // find access without users
genericOrphanCheck("Access entries without existing user", genericOrphanCheck("Access entries without existing user",
"access", "user", "access.user_id=user.id"), "access", "user", "access.user_id=`user`.id"),
// find access without repository // find access without repository
genericOrphanCheck("Access entries without existing repository", genericOrphanCheck("Access entries without existing repository",
"access", "repository", "access.repo_id=repository.id"), "access", "repository", "access.repo_id=repository.id"),