Change host lookup to be optional #23

Georg · 2024-10-02T22:35:15+02:00

Georg commented

2024-10-02 22:35:15 +02:00

Some parts of the logic do not make use of the host column in the players table, allow the field to be empty and do not query for an empty value (which might return bogus entries) if no value is passed to the lookup function. This additionally avoids the need for the hardcoded initial host when configuring the bot player.

Signed-off-by: Georg Pfuetzenreuter mail@georg-pfuetzenreuter.net

Some parts of the logic do not make use of the host column in the players table, allow the field to be empty and do not query for an empty value (which might return bogus entries) if no value is passed to the lookup function. This additionally avoids the need for the hardcoded initial host when configuring the bot player. Signed-off-by: Georg Pfuetzenreuter <mail@georg-pfuetzenreuter.net>

Georg added 1 commit 2024-10-02 22:35:16 +02:00

Change host lookup to be optional accf16c92a

Some parts of the logic do not make use of the host column in the
players table, allow the field to be empty and do not query for an empty
value (which might return bogus entries) if no value is passed to the
lookup function.
This additionally avoids the need for the hardcoded initial host when
configuring the bot player.

Signed-off-by: Georg Pfuetzenreuter <mail@georg-pfuetzenreuter.net>

pratyush merged commit b6187e0077 into master

2024-10-03 18:01:00 +02:00

pratyush referenced this issue from a commit

2024-10-03 18:01:00 +02:00

Merge pull request 'Change host lookup to be optional' (#23) from dbhost into master

pratyush deleted branch dbhost

2024-10-03 18:01:00 +02:00

Sign in to join this conversation.

No reviewers