Skip to content

Conversation

418Coffee
Copy link
Contributor

No description provided.

@rueian
Copy link
Collaborator

rueian commented Apr 22, 2022

Hi @418Coffee, this branch needs to be rebased, otherwise I am not able to merge it.

I also noticed that you have some wrong usage of Arbitrary command builder.

For example, the c.client.B().Arbitrary("XADD", a.Stream) should be c.client.B().Arbitrary("XADD").Keys(a.Stream)

@418Coffee
Copy link
Contributor Author

Should be good to go now?

@rueian rueian merged commit 38e4982 into redis:rueidiscompat-2 Apr 22, 2022
@rueian
Copy link
Collaborator

rueian commented Apr 22, 2022

Yes! Thanks a lot!

rueian pushed a commit that referenced this pull request Dec 29, 2024
Add `skip_verify` or `skip_verify=true` to disable TLS certificate
verification. Default is false.

Inspired by various Go drivers:
- ClickHouse: https://github.com/ClickHouse/clickhouse-go/blob/v2.30.0/clickhouse_options.go#L259
- MongoDB: https://github.com/mongodb/mongo-go-driver/blob/v2.0.0/x/mongo/driver/connstring/connstring.go#L609
- MySQL: https://github.com/go-sql-driver/mysql/blob/v1.8.1/dsn.go#L175

Signed-off-by: Julien Riou <julien@riou.xyz>
Signed-off-by: Rueian <rueiancsie@gmail.com>
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