Conversation
WalkthroughThis pull request adds Crystal driver documentation to CrateDB, introducing a new grid card in the drivers index and a comprehensive Crystal-specific connection guide with installation instructions, code examples, and SSL configuration details. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
docs/connect/crystal/index.md
Outdated
| :::{rubric} CrateDB Cloud | ||
| ::: | ||
|
|
||
| For connecting to CrateDB Cloud, use `sslmode=require&auth_methods=cleartext` | ||
| parameters, and replace username, password and hostname with values matching | ||
| your environment. | ||
| ```crystal | ||
| DB.open("postgres://admin:password@testcluster.cratedb.net:5432/doc?sslmode=require&auth_methods=cleartext") | ||
| ``` |
There was a problem hiding this comment.
Problem
When not permitting auth_methods=cleartext, the connection will fail, because cleartext is disabled by default.
Unhandled exception: (DB::ConnectionRefused)
Caused by: server asked for disabled authentication method: cleartext (PQ::ConnectionError)Background
By default this driver will accept
scram-sha-256andmd5, as well as
trust. Howevercleartextis disabled by default. You can control exactly
which auth methods the client will accept by passing in a comma separated list
to theauth_methodsparameter.
Thoughts
Do you think it is applicable to report this to crate/crate, so it may possibly support md5 or scram-sha-256 in the future?
There was a problem hiding this comment.
I've created a ticket about this, so we can proceed.
712b0cb to
c3c3791
Compare
kneth
left a comment
There was a problem hiding this comment.
Let's discuss https://github.com/crate/tech-content/issues/124#issuecomment-3485676563 before merging
About
What the title says.
Preview
https://cratedb-guide--421.org.readthedocs.build/connect/crystal/