Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Datatabase schema is not set #34

Open
frafra opened this issue Apr 20, 2021 · 1 comment
Open

Datatabase schema is not set #34

frafra opened this issue Apr 20, 2021 · 1 comment

Comments

@frafra
Copy link
Contributor

frafra commented Apr 20, 2021

Database schema is required by this script:

dbSchema = config.getProperty('Database', 'schema')

Missing from:

[Database]
; General DB properties
name = ailabeltooldb
host = localhost
port = 17685
user = ailabeluser
password = aiLabelUser
; Maximum number of (concurrent) connections per server instance.
; Under normal circumstances, this does not need to be altered. You may set this to a higher value,
; at the cost of having potentially many TCP connections to the database server open.
; Just make sure to set it high enough for multiple services to be able to access the DB at a time.
max_num_connections = 16

I would also suggest raising an error if the parameter is mandatory but missing.

schema should be equal to the user name.

frafra added a commit to frafra/aerial_wildlife_detection that referenced this issue Apr 20, 2021
@bkellenb
Copy link
Collaborator

Thank you very much for this issue.
This is a legacy script that originates to AIDE version 1.0 where only one "project" could be supported at a time (and had to be specified in the settings.ini file, precisely through the schema keyword). With AIDE version 2.0, this keyword has become obsolete as an arbitrary number of projects can now be created.

I have resolved the issue in the latest commit (and also removed the SQL injection vulnerability mentioned in the respective issue). The same holds for the file in the latest development branch.

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 a pull request may close this issue.

2 participants