Skip to content

Conversation

@mickael-plassard
Copy link
Contributor

@mickael-plassard mickael-plassard commented Aug 13, 2025

Copy link
Contributor

@mcimbaro mcimbaro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMO the PR title can be made more explicit :)

title: Accessing Your Scalingo for MySQL® Addon
nav: Accessing
modified_at: 2025-06-10 12:00:00
modified_at: 2025-08-13 14:50:00
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nitpick: We do not change the hour in general, but only the date

Suggested change
modified_at: 2025-08-13 14:50:00
modified_at: 2025-08-13 12:00:00

Comment on lines 304 to 306
Receiving the error `Too many connection errors from 1.2.3.4:12345` means
that the MySQL router blocked the connection after too many attempts were
tried using an invalid configuration.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion: I find the current sentence hard to understand

Suggested change
Receiving the error `Too many connection errors from 1.2.3.4:12345` means
that the MySQL router blocked the connection after too many attempts were
tried using an invalid configuration.
Receiving the error `Too many connection errors from 1.2.3.4:12345` means
that the MySQL router has blocked the connection after multiple failed
attempts using invalid credentials.

@mickael-plassard mickael-plassard changed the title fix #2881 fix(mysql): document too many connection error Aug 13, 2025
@Frzk
Copy link
Contributor

Frzk commented Aug 14, 2025

Hi 👋
Thanks for caring about the doc ❤️
Would it be possible to move this to the Troubleshooting page? 🙏

@mickael-plassard mickael-plassard self-assigned this Aug 14, 2025
@mickael-plassard mickael-plassard requested a review from Frzk August 14, 2025 15:35
@Frzk Frzk self-requested a review September 19, 2025 16:30
Copy link
Contributor

@Frzk Frzk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

4 participants