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

[ENH]: Test SecureChat Application with MySQL and Fix Compatibility Issues #161

Open
1 task done
pravin-bansod opened this issue Dec 18, 2024 · 1 comment
Open
1 task done
Labels
enhancement New feature or request

Comments

@pravin-bansod
Copy link
Contributor

Contact Details

[email protected]

Feature Description

The SecureChat application supports RDS databases but has only been tested with SQLite. We need to:

  • Test the application with MySQL.
  • Identify and resolve any compatibility issues.

Quick attempt results below errors on startup -
root@24c1a05ee031:/# paig_securechat run --port 2535 --host 0.0.0.0 INFO [alembic.runtime.migration] Context impl MySQLImpl. INFO [alembic.runtime.migration] Will assume non-transactional DDL. INFO [alembic.runtime.migration] Context impl MySQLImpl. INFO [alembic.runtime.migration] Will assume non-transactional DDL. INFO [alembic.runtime.migration] Running upgrade -> 3f8e6786bde9, Initial migration An error occurred during tables creation on database: (in table 'conversations', column 'client_app_id'): VARCHAR requires a length on dialect mysql An error occurred during tables creation on database: (in table 'conversations', column 'client_app_id'): VARCHAR requires a length on dialect mysql

Tasks:

  • Set up a MySQL database for testing.
  • Update configuration as needed for MySQL.
  • Run tests and identify compatibility issues.
  • Fix identified issues to ensure MySQL compatibility.
  • Document the steps for using MySQL with SecureChat.

Acceptance Criteria:

  • Application works seamlessly with MySQL.
  • No critical errors or data inconsistencies.
  • Documentation updated for MySQL configuration.

Code of Conduct

  • I agree to follow this project's Code of Conduct
@pravin-bansod pravin-bansod added the enhancement New feature or request label Dec 18, 2024
@amrakshay
Copy link
Contributor

I was able to get the SecureChat up and running with MySQL database, though I have not evaluated through all the use cases. Whoever taking this issue to work on can refer below.

  • Patch for the changes - These changes include data type changes, so it should support both SQLite and MySQL
    paig-securechat-mysql-support.patch
  • Dependencies - Also you will need to install below libraries as mysql driver for connection
pip install mysql-connector-python
pip install asyncmy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants