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

Update Connection.py #713

Open
wants to merge 86 commits into
base: sqla
Choose a base branch
from
Open

Update Connection.py #713

wants to merge 86 commits into from

Conversation

Ishanv5
Copy link

@Ishanv5 Ishanv5 commented Mar 31, 2024

Added More Field Type like sets , enums , binary and char

methane and others added 30 commits October 20, 2021 16:34
There is a small typo in tests/dbapi20.py.

Should read `portable` rather than `portible`.
Co-authored-by: LGTM Migrator <lgtm-migrator@users.noreply.github.com>
Use MariaDB Connector/C 3.3.4
MySQL breaks mysql_config often. Use pkg-config instead.

Fixes PyMySQL#584
* Drop Python 3.7
* Use latest actions

Fix PyMySQL#591
methane and others added 30 commits June 23, 2023 05:05
fixed typo in Customize build (POSIX)
MySQL 8.0.33+ shows deprecation warning to stderr.
So we avoid using it as possible.

In the future, we will deprecate `reconnect` option of the
`Connection.ping()`.
homebrew's prefix can vary - with this minor change, the pkgconfig path
should work in more user environments.
MySQL use ssl by default but MariaDB don't.

Until mysqlclient<=2.2.1, `ssl=True` unintentionally allowed and it
called `mysql_ssl_set(mysql, NULL, NULL, NULL, NULL, NULL)`. Although it
is no-op in MySQL Connector, MariaDB Connector silently set
MYSQL_OPT_SSL_ENFORCE when the API is called. (See PyMySQL#698)

In case of PyMySQL, ssl is not used by default but `ssl=True` behave
like `sslmode="PREFERRED"`.

For better backward compatibility and compatibility with PyMySQL and
security, I decided to allow ssl=True and it means sslmode="REQUIRED" on
MySQL Connector and
set MYSQL_OPT_SSL_ENFORCE on MariaDB Connector.

Fix PyMySQL#699
Added New Field type char , set ,enums,Binary
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.