Skip to content

style: use no-secure option instead of secure option#2

Open
nicecui wants to merge 1 commit into
mainfrom
user-pass-optional
Open

style: use no-secure option instead of secure option#2
nicecui wants to merge 1 commit into
mainfrom
user-pass-optional

Conversation

@nicecui

@nicecui nicecui commented Nov 13, 2023

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread app.go
var username = flag.String("username", "", "The username of the database.")
var password = flag.String("password", "", "The password of the database.")
var secure = flag.Bool("secure", true, "Whether to use secure connection to GreptimeDB. `true` or `false`. Default is `true`.")
var noSecure = flag.Bool("no-secure", false, "Do not use secure connection to GreptimeDB. `true` or `false`. Default is `false`.")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

insecure

@sunng87

sunng87 commented Nov 13, 2023

Copy link
Copy Markdown
Contributor

Javascript and python are using secure, why we changed it for go only?

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 this pull request may close these issues.

2 participants