-
Notifications
You must be signed in to change notification settings - Fork 2
Made password configurable for tranSMART database. #41
base: master
Are you sure you want to change the base?
Conversation
@TG908 Hi! Add this information to README. |
@ewelinagr will this require a change for entrypoint script of transmart container image? |
It looks like the password is set in here but never used. |
Indeed, first we need to change transmart-api-server entrypoint to make this working. Here is a link to track the issue on transmart-api-server side: thehyve/transmart-core#472. |
I've made the necessary changes: thehyve/transmart-core#473 |
any updates on this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree that the password should be configurable. Please rename the variable to BIOMART_USER_PASSWORD
to be consistent with the TranSMART database user/schema names.
Just a general remark: this user has super user rights on the database. So, it is better to not expose the database port by default, but only accessible for administrators, e.g., not exposing the port to outside the host machine, but only through an SSH tunnel. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you squash your commits into a single one?
5575798
to
408665f
Compare
done |
Since the tranSMART Database will be exposed by port forwarding it makes sense to make the password user configurable.