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

Configure nginx to send 2.5 clients to 2.6 lobby #11191

Merged
merged 1 commit into from
Nov 6, 2022
Merged

Conversation

DanVanAtta
Copy link
Member

This update would have NGINX (which is listening on 'prod.triplea-game.org') send 2.5 game clients to the 2.6 lobby (2.5 is the current 'main' release).

The 2.6 lobby and 2.5 game clients are compatible with each other, so this is a safe configuration update to make.

Though, 2.5 clients are currently configured to use a 'prod2' URL and they would see no difference. But, we can configure the 2.5 clients to start using the 'prod.triplea-game' server by modifying the lobby_games.yml file, which would then cause 2.5 clients to be redirect to the 2.6 lobby. Until then, this change has no impact.

Change Summary & Additional Notes

Release Note

This update would have NGINX (which is listening on 'prod.triplea-game.org')
send 2.5 game clients to the 2.6 lobby (2.5 is the current 'main' release).

The 2.6 lobby and 2.5 game clients are compatible with each other, so this
is a safe configuration update to make.

Though, 2.5 clients are currently configured to use a 'prod2' URL and they
would see no difference. But, we can configure the 2.5 clients to start
using the 'prod.triplea-game' server by modifying the lobby_games.yml file,
which would then cause 2.5 clients to be redirect to the 2.6 lobby. Until
then, this change has no impact.
@codecov
Copy link

codecov bot commented Nov 6, 2022

Codecov Report

Merging #11191 (c248fdd) into master (e31fd9e) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #11191   +/-   ##
=========================================
  Coverage     27.21%   27.22%           
  Complexity     8250     8250           
=========================================
  Files          1217     1217           
  Lines         78074    78077    +3     
  Branches      10615    10615           
=========================================
+ Hits          21251    21255    +4     
+ Misses        54855    54854    -1     
  Partials       1968     1968           
Impacted Files Coverage Δ
...lea/ai/flowfield/influence/InfluenceTerritory.java 96.96% <0.00%> (-3.04%) ⬇️
...c/main/java/org/triplea/swing/SwingComponents.java 4.08% <0.00%> (-0.09%) ⬇️
.../main/java/games/strategy/net/ClientMessenger.java 64.58% <0.00%> (+2.08%) ⬆️
.../src/main/java/games/strategy/net/nio/Decoder.java 79.31% <0.00%> (+5.17%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@DanVanAtta DanVanAtta merged commit 7c526ee into master Nov 6, 2022
@DanVanAtta DanVanAtta deleted the nginx-config branch November 6, 2022 04:52
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.

None yet

1 participant