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

Pass DB connection details using file instead of env. variables #6353

Merged
merged 1 commit into from
Sep 6, 2023

Conversation

thurka
Copy link
Contributor

@thurka thurka commented Aug 17, 2023

This PR uses temporaty file to pass DB connection details. The file is created in subdirectory of OS temporaty directory and has only minimal (read) permission for user running NBLS.Other system users does not have even read permission (this is similar to ssh private key files. If the setting of correct permissions fails, the file the temporary file is deleted immediatelly. The temporaty file is deleted as soon as debugging session ends. If the NBLS and VScode are hard killed and the temporary file remains on filesystem, it is deleted on the subsequent startup of NBLS.

@thurka thurka added LSP [ci] enable Language Server Protocol tests VSCode Extension [ci] enable VSCode Extension tests labels Aug 17, 2023
@thurka thurka self-assigned this Aug 17, 2023
@thurka thurka added the do not merge Don't merge this PR, it is not ready or just demonstration purposes. label Aug 17, 2023
@thurka thurka changed the title Pass DB connection details using file instead of env. variables WIP: Pass DB connection details using file instead of env. variables Aug 17, 2023
@thurka thurka force-pushed the GCN-3053 branch 6 times, most recently from 5daba6f to c9d4e42 Compare August 29, 2023 15:40
@thurka thurka changed the title WIP: Pass DB connection details using file instead of env. variables Pass DB connection details using file instead of env. variables Sep 4, 2023
@thurka thurka removed the do not merge Don't merge this PR, it is not ready or just demonstration purposes. label Sep 4, 2023
Copy link
Contributor

@jhorvath jhorvath left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@thurka thurka merged commit fef90a1 into apache:master Sep 6, 2023
34 checks passed
@thurka thurka deleted the GCN-3053 branch September 6, 2023 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LSP [ci] enable Language Server Protocol tests VSCode Extension [ci] enable VSCode Extension tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants