Skip to content

fix(net): prefer TLS for getConfiguration requests - #4104

Open
rafaadeveloper wants to merge 1 commit into
citizenfx:masterfrom
rafaadeveloper:fix/4064-getconfiguration-tls
Open

fix(net): prefer TLS for getConfiguration requests#4104
rafaadeveloper wants to merge 1 commit into
citizenfx:masterfrom
rafaadeveloper:fix/4064-getconfiguration-tls

Conversation

@rafaadeveloper

Copy link
Copy Markdown

Goal of this PR

Make getConfiguration prefer HTTPS instead of always using plaintext HTTP when connecting to the game endpoint.

This removes the mandatory plaintext path for the configuration request while preserving compatibility with servers that may still require the legacy HTTP behavior.

How is this PR achieving the goal

  • Attempts to fetch getConfiguration over HTTPS first.
  • Falls back to the existing HTTP path only when the TLS request fails at the transport level.
  • Preserves the existing request payload and response handling.
  • Avoids masking valid HTTP responses or server-side errors by only retrying when no HTTP response was received.

This PR applies to the following area(s)

  • FiveM
  • Networking
  • Client connection flow

Successfully tested on

Game builds: Not tested

Platforms: Not tested

Checklist

  • Code compiles and has been tested successfully.
  • Code explains itself well and/or is documented.
  • My commit message explains what the changes do and what they are for.
  • No extra compilation warnings are added by these changes.

Fixes issues

Addresses #4064

@github-actions github-actions Bot added the invalid Requires changes before it's considered valid and can be (re)triaged label Aug 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

invalid Requires changes before it's considered valid and can be (re)triaged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant