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

ClientInfo property not supported. #160

Closed
KKould opened this issue Mar 3, 2024 · 10 comments
Closed

ClientInfo property not supported. #160

KKould opened this issue Mar 3, 2024 · 10 comments

Comments

@KKould
Copy link
Contributor

KKould commented Mar 3, 2024

I tried to run the Sqlite in Example and connect using the database gui in IDEA, but it prompted ClientInfo property not supported.

How should I make it support this?

image

ref: #99

@sunng87
Copy link
Owner

sunng87 commented Mar 4, 2024

@KKould Thank you for reporting this. I don't have an IDEA so I may need more information from you.

  1. Did you seen any panic from the server side?
  2. Could you please use wireshark to capture the traffic when you do connection test?

@KKould
Copy link
Contributor Author

KKould commented Mar 4, 2024

@KKould Thank you for reporting this. I don't have an IDEA so I may need more information from you.

  1. Did you seen any panic from the server side?
  2. Could you please use wireshark to capture the traffic when you do connection test?
  1. After fixing it, the error will no thrown. ref: fix(example/sqlite): use Type::UNKNOWN when Column::dec_type returns None. #163
  2. I tried using WireShake but it looks like there is no message

Tips: This is my first time using this, maybe I'm using it incorrectly
image

@sunng87
Copy link
Owner

sunng87 commented Mar 4, 2024

Thank you! Are you capturing the right net interface? If your server is listening to localhost, you need to capture loopback.

@KKould
Copy link
Contributor Author

KKould commented Mar 4, 2024

Oh, it seems I chose the wrong network card. I exported this file from wireshark.

sqlite.json

@sunng87
Copy link
Owner

sunng87 commented Mar 5, 2024

Fixed in #163

@sunng87 sunng87 closed this as completed Mar 5, 2024
@KKould
Copy link
Contributor Author

KKould commented Mar 5, 2024

@sunng87 There were some misunderstandings. After I used this pr to fix them, I just prevented the server from throwing errors, but it still prompted ClientInfo property not supported.

@sunng87
Copy link
Owner

sunng87 commented Mar 5, 2024

Sorry, let me reopen this. I will find time to analyse your captured packets.

@sunng87 sunng87 reopened this Mar 5, 2024
@sunng87
Copy link
Owner

sunng87 commented Mar 9, 2024

From the paccap I saw IDEA sends a query select version() via extended subprotocol under the hood. This query is not supported by sqlite so it does not reply a result that IDEA expects.

@KKould
Copy link
Contributor Author

KKould commented Mar 10, 2024

From the paccap I saw IDEA sends a query select version() via extended subprotocol under the hood. This query is not supported by sqlite so it does not reply a result that IDEA expects.

Is there a way for sqlite to support it?

@sunng87
Copy link
Owner

sunng87 commented Mar 11, 2024

We don't have to support it in sqlite. You can add a layer on top of statement::query to process these queries out of sqlite. If you are interested in this feature, a PR is welcomed. Also you may want to do the same thing on duckdb example.

@KKould KKould closed this as completed Mar 11, 2024
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

No branches or pull requests

2 participants