Skip to content

feat: SSL/TLS implementation for require_secure_transport#3558

Merged
Helias merged 17 commits into
azerothcore:masterfrom
TheSCREWEDSoftware:secure_connection_2
Oct 6, 2025
Merged

feat: SSL/TLS implementation for require_secure_transport#3558
Helias merged 17 commits into
azerothcore:masterfrom
TheSCREWEDSoftware:secure_connection_2

Conversation

@TheSCREWEDSoftware

Copy link
Copy Markdown
Contributor

This implementation allows people that have require_secure_transport enabled.

Closes: #3556

#3556 (comment)

Being tested by the issue author also.

I tried following the current config sturcutre that's being used, I'm unsure about the description for the SSL/TLS, im just thinking of having it without a description, open to feedback.

This can be tested if you want on your side by using your super user for mysql and running the following:

SET GLOBAL require_secure_transport = ON;

And having TLS option off.
Image

And having TLS option on

Image

@TheSCREWEDSoftware

Copy link
Copy Markdown
Contributor Author

CC @FrancescoBorzi and @Helias

@TheSCREWEDSoftware

Copy link
Copy Markdown
Contributor Author

This I may need help fixing, as I'm unsure now from the error, it does build fine in windows and works

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Comment thread libs/shared/login-config/src/login-config.service.ts Outdated
Comment thread libs/main/connection-window/src/connection-window.component.ts Outdated
dependabot Bot and others added 7 commits September 27, 2025 22:24
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@TheSCREWEDSoftware

Copy link
Copy Markdown
Contributor Author

// SET GLOBAL require_secure_transport = ON;

Unchecked = Error, intended
image

Checked = No error, intended
image

And thanks helias for adjusting it to the Keira standards

Comment thread .github/workflows/release.yml Outdated
needs: get-package-version
name: Build and upload the Mac release assets
runs-on: macos-13 # it's the most recent version still running on Intel, we keep it for retro-compatibility
runs-on: macos-15 # it's the most recent version still running on Intel, we keep it for retro-compatibility

@FrancescoBorzi FrancescoBorzi Sep 29, 2025

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this does not seem right (also should be addressed in a separate PR, not here):

Image

https://docs.github.com/en/actions/reference/runners/github-hosted-runners

the rest I will check soon! thanks @TheSCREWEDSoftware <3

Comment thread libs/main/connection-window/src/connection-window.component.ts Outdated
Comment thread libs/main/connection-window/src/connection-window.component.html Outdated
@FrancescoBorzi FrancescoBorzi changed the title SSL/TLS implementation for require_secure_transport feat: SSL/TLS implementation for require_secure_transport Sep 29, 2025
@TheSCREWEDSoftware

TheSCREWEDSoftware commented Sep 29, 2025

Copy link
Copy Markdown
Contributor Author
image image

For the other question, I can't answer, i leave it to be answered by helias :p

@TheSCREWEDSoftware

Copy link
Copy Markdown
Contributor Author

I swear to god i wish keira worked with github desktop so i dont have to use the git cli which sometimes is stupid

@TheSCREWEDSoftware

Copy link
Copy Markdown
Contributor Author

Can test this during the afternoon

@TheSCREWEDSoftware

Copy link
Copy Markdown
Contributor Author

All working

@Helias Helias merged commit 7160b38 into azerothcore:master Oct 6, 2025
15 of 16 checks passed
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.

Connection to MySQL Fails When Secure Transport is Enabled

3 participants