Skip to content

Accept DSN schemes with a plus sign#1979

Merged
rolandwalker merged 1 commit into
mainfrom
RW/accept-dsn-schemes-with-plus
Jul 3, 2026
Merged

Accept DSN schemes with a plus sign#1979
rolandwalker merged 1 commit into
mainfrom
RW/accept-dsn-schemes-with-plus

Conversation

@rolandwalker

Copy link
Copy Markdown
Contributor

Description

DSNs are sometimes written in the form

mysql+pymysql://mycli@localhost:3306/mysql

with extensions to the scheme. While we will ignore the part of the scheme to the right of the plus sign, mycli should accept the DSN and make the connection.

Checklist

  • I added this contribution to the changelog.md file.
  • I added my name to the AUTHORS file (or it's already there).
  • To lint and format the code, I ran
    uv run ruff check && uv run ruff format && uv run mypy --install-types .

DSNs are sometimes written in the form

    mysql+pymysql://mycli@localhost:3306/mysql

with extensions to the scheme.  While we will ignore the part of the
scheme to the right of the plus sign, mycli should accept the DSN and
make the connection.
@rolandwalker rolandwalker self-assigned this Jul 3, 2026
@rolandwalker rolandwalker merged commit 85ed95b into main Jul 3, 2026
11 checks passed
@rolandwalker rolandwalker deleted the RW/accept-dsn-schemes-with-plus branch July 3, 2026 16:55
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.

1 participant