#169 — [Bug]: Old ASQLDatabase converts PG schema and role with upper-case characters

Repo: Twill-AI/twill-llm-engine State: closed | Status: done Assignee: meliascosta

Created: 2024-10-21 · Updated: 2024-10-24

Description

Steps to reproduce

To reproduce:

  1. In staging create tenant with name containing upper-cased characters. For this need to enter “Organization name” of length 10 characters with upper-cased letters (POST /self-management/tenant in API).
  2. After adding integration try to ask LlmEngine about any business-data aspect.

What was the expected behavior?

I expected it to answer.

Screenshots

image

Facade logs
2024-10-21T08:23:25.613383143Z INFO:     ('10.0.2.51', 53876) - "WebSocket /api/v1/ws/c5728be6-4aee-4747-a0cd-cadcbe11e204" [accepted]
2024-10-21T08:23:25.613706321Z INFO:     connection open
2024-10-21T08:23:25.652029016Z 2024-10-21 08:23:25.651|INFO|AlksndrGgl|database:65|Pool already exists
2024-10-21T08:23:26.143910085Z 2024-10-21 08:23:26.143|INFO|AlksndrGgl|_client:1729|HTTP Request: POST https://twill-east.openai.azure.com//openai/deployments/twill-gpt4o/chat/completions?api-version=2024-07-01-preview "HTTP/1.1 200 OK"
2024-10-21T08:23:27.065693817Z 2024-10-21 08:23:27.065|INFO|AlksndrGgl|database:65|Pool already exists
2024-10-21T08:23:27.318764617Z 2024-10-21 08:23:27.318|INFO|AlksndrGgl|_client:1729|HTTP Request: POST https://twill-east.openai.azure.com//openai/deployments/twill-gpt4o/chat/completions?api-version=2024-07-01-preview "HTTP/1.1 200 OK"
2024-10-21T08:23:27.952896887Z 2024-10-21 08:23:27.952|INFO|AlksndrGgl|

## Notes

_Add implementation notes, blockers, and context here_

## Related

_Add wikilinks to related people, meetings, or other tickets_