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

620.bugfix program_name default value #621

Closed

Conversation

mrlucasrib
Copy link

@mrlucasrib mrlucasrib commented Oct 5, 2021

What do these changes do?

When the program path is too big, an error is thrown because it overflows the byte limit of a request to mysql. This change fixes this defining that only the last item of the path is the name of the program.

Are there changes in behavior for the user?

When the program name includes its full path, it extracts only the program name.

Related issue number

#620

Checklist

  • I think the code is well written
  • Unit tests for the changes exist
  • Documentation reflects the changes
  • Add a new news fragment into the CHANGES folder

When the program path is too big, an error is thrown because it overflows the byte limit of a request to mysql. This change fixes this defining that only the last item of the path is the name of the program.
@Nothing4You Nothing4You linked an issue Jan 12, 2022 that may be closed by this pull request
@Nothing4You
Copy link
Collaborator

It is probably a better idea to just port the change from PyMySQL and not send this by default: PyMySQL/PyMySQL#755

@Nothing4You Nothing4You added this to the 1.0.0 milestone Jan 16, 2022
@Nothing4You
Copy link
Collaborator

do you want to adjust your PR to port this change?
otherwise I'll do this myself soonish.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error message by program name too big
2 participants