TripleA development
-
@cernel And if one would have gone ahead making the proposed changes without checking the rules very carefully, I would say chances are doing more bad than good.
On top of that, you must also consider supporting custom games. TripleA is not all about un-original work.
-
@panther said in TripleA development:
Of course all
Can I assume that the list in https://forums.triplea-game.org/topic/1549/rules-issues-with-the-triplea-engine is maintained well and they are meant to have all the same label "Impact: Bad Game Rules" on github?
If so, can you help to prioritize them?I absolutely understand the frustration that is going around. However, I would hope that you all are still willing to assist in
(1) structuring the issues we have
(2) organize ourselves, so responsibilities and current status,
so we can get a clear overview and try to tackle the issues according to their priority.
My hope would be that we get to a point where we can onboard new developers (responsibility with the dev team) and hand them a sorted (!) list of issues (responsibilities lies with moderators assisted by the end users) from which they can choose and provide fixes. -
@magicstyck Why is there such a big deal about changes breaking saved games? If you start game, keep using the same version until you are done.
Breaking existing XML is more of an issue, but we have done it before, and I personally fixed many mods with a simple batch file.
So please, feel free to go ahead with refactoring code and don't worry about covering every obscure A&A rules case.
What does need to be done to discuss major changes before they are made. 2.6 did major changes like deprecating mapName and Notes with little discussion.
-
@frigoref said in TripleA development:
Can I assume that the list in https://forums.triplea-game.org/topic/1549/rules-issues-with-the-triplea-engine is maintained well and they are meant to have all the same label "Impact: Bad Game Rules" on github?
If so, can you help to prioritize them?In general you can assume that. What I don't know is - as PR do not always refer to issues - whether working on other details maybe had any impact on them. What I will do is check the labels again - and of course I will help to proritize them. There are serious issues and minor issues, that's for sure.
-
IMHO we should not push forward 2.6 as long as major issues such as the MARTI-issue and the broken-PBF-nodeBB-issue are not fixed.
-
@panther can you please also label the issues you consider 2.6 blocking issues?
Do you need help for this?Is there anyone interested in assisting panter?
-
-
I've been looking at the FeignException 404 on map listing. I know what is happening and why. I'm working on a test and coding a fix for it. I should have a PR in by the end of the weekend. I can help work on any other blockers after that.
All related issues:
https://github.com/triplea-game/triplea/issues/10023
https://github.com/triplea-game/triplea/issues/10053
https://github.com/triplea-game/triplea/issues/10079 -
@magicstyck sounds great! Many thanks for taking this one up!
-
@magicstyck Let me know if you can get to the point that mods can be downloaded using 2.6. For now, I am doing my updates to the map repository using 2.5.
-
@frigoref @Panther @RogerCooper @LaFayette
@magicstyck said in TripleA development:
I've been looking at the FeignException 404 on map listing. I know what is happening and why. I'm working on a test and coding a fix for it. I should have a PR in by the end of the weekend. I can help work on any other blockers after that.
When I posted this afternoon, I thought the issue was a broken endpoint in the spitfire-server, the code that runs the lobby. I have since confirmed locally, that is not the case. When I spin up the 2.6 lobby, I'm able to run a GET request on the /maps/listing endpoint without issue.
curl -i -H "Accept: application/json" -H "Content-Type: application/json" -H "Authorization: Bearer oijsad80lksjdflsdflk" -H "System-Id-Header: 1234567890" -H "Triplea-Version: 2.6" -v http://127.0.0.1:8080/maps/listing * Trying 127.0.0.1... * TCP_NODELAY set * Connected to 127.0.0.1 (127.0.0.1) port 8080 (#0) > GET /maps/listing HTTP/1.1 > Host: 127.0.0.1:8080 > User-Agent: curl/7.64.1 > Accept: application/json > Content-Type: application/json > Authorization: Bearer oijsad80lksjdflsdflk > System-Id-Header: 1234567890 > Triplea-Version: 2.6 > < HTTP/1.1 200 OK HTTP/1.1 200 OK < Date: Thu, 17 Feb 2022 05:39:42 GMT Date: Thu, 17 Feb 2022 05:39:42 GMT < Content-Type: application/json Content-Type: application/json < Vary: Accept-Encoding Vary: Accept-Encoding < Content-Length: 214 Content-Length: 214 < * Connection #0 to host 127.0.0.1 left intact [{"downloadUrl":"http://download-url","previewImageUrl":"http://preview-image","mapName":"test-map","lastCommitDateEpochMilli":1644988237583,"description":"map description","mapTags":[],"downloadSizeInBytes":1024}]* Closing connection 0I logged into the postgres database locally and deleted the record in the map_index table of the lobby_db (where data returned above comes from). This confirmed that if the table is empty, the /maps/listing endpoint does not return a 404. It returns a 200 and an empty JSON array.
curl -i -H "Accept: application/json" -H "Content-Type: application/json" -H "Authorization: Bearer oijsad80lksjdflsdflk" -H "System-Id-Header: 1234567890" -H "Triplea-Version: 2.6" -v http://127.0.0.1:8080/maps/listing * Trying 127.0.0.1... * TCP_NODELAY set * Connected to 127.0.0.1 (127.0.0.1) port 8080 (#0) > GET /maps/listing HTTP/1.1 > Host: 127.0.0.1:8080 > User-Agent: curl/7.64.1 > Accept: application/json > Content-Type: application/json > Authorization: Bearer oijsad80lksjdflsdflk > System-Id-Header: 1234567890 > Triplea-Version: 2.6 > < HTTP/1.1 200 OK HTTP/1.1 200 OK < Date: Thu, 17 Feb 2022 05:45:03 GMT Date: Thu, 17 Feb 2022 05:45:03 GMT < Content-Type: application/json Content-Type: application/json < Vary: Accept-Encoding Vary: Accept-Encoding < Content-Length: 2 Content-Length: 2 < * Connection #0 to host 127.0.0.1 left intact []* Closing connection 0According to closed PR that introduced the change (https://github.com/triplea-game/triplea/pull/9940), if I pass the Triplea-Version header then nginx should proxy me to the correct lobby. If the new nginx config is deployed, then it should read that header key and send me to the 2.6 lobby. If it isn't, then it will send me to the 2.5 lobby on every request. I sent the key with a value of 2.6 and was returned a 404.
curl -i -H "Accept: application/json" -H "Content-Type: application/json" -H "Authorization: Bearer oijsad80lksjdflsdflk" -H "System-Id-Header: 1234567890" -H "Triplea-Version: 2.6" -v https://prod2-lobby.triplea-game.org/maps/listing * Trying 173.255.225.250... * TCP_NODELAY set * Connected to prod2-lobby.triplea-game.org (173.255.225.250) port 443 (#0) * ALPN, offering h2 * ALPN, offering http/1.1 * successfully set certificate verify locations: * CAfile: /etc/ssl/cert.pem CApath: none * TLSv1.2 (OUT), TLS handshake, Client hello (1): * TLSv1.2 (IN), TLS handshake, Server hello (2): * TLSv1.2 (IN), TLS handshake, Certificate (11): * TLSv1.2 (IN), TLS handshake, Server key exchange (12): * TLSv1.2 (IN), TLS handshake, Server finished (14): * TLSv1.2 (OUT), TLS handshake, Client key exchange (16): * TLSv1.2 (OUT), TLS change cipher, Change cipher spec (1): * TLSv1.2 (OUT), TLS handshake, Finished (20): * TLSv1.2 (IN), TLS change cipher, Change cipher spec (1): * TLSv1.2 (IN), TLS handshake, Finished (20): * SSL connection using TLSv1.2 / ECDHE-RSA-AES256-GCM-SHA384 * ALPN, server accepted to use h2 * Server certificate: * subject: CN=prod2-lobby.triplea-game.org * start date: Jan 30 23:55:24 2022 GMT * expire date: Apr 30 23:55:23 2022 GMT * subjectAltName: host "prod2-lobby.triplea-game.org" matched cert's "prod2-lobby.triplea-game.org" * issuer: C=US; O=Let's Encrypt; CN=R3 * SSL certificate verify ok. * Using HTTP2, server supports multi-use * Connection state changed (HTTP/2 confirmed) * Copying HTTP/2 data in stream buffer to connection buffer after upgrade: len=0 * Using Stream ID: 1 (easy handle 0x7ff154810000) > GET /maps/listing HTTP/2 > Host: prod2-lobby.triplea-game.org > User-Agent: curl/7.64.1 > Accept: application/json > Content-Type: application/json > Authorization: Bearer oijsad80lksjdflsdflk > System-Id-Header: 1234567890 > Triplea-Version: 2.6 > * Connection state changed (MAX_CONCURRENT_STREAMS == 128)! < HTTP/2 404 HTTP/2 404 < server: nginx/1.18.0 (Ubuntu) server: nginx/1.18.0 (Ubuntu) < date: Thu, 17 Feb 2022 06:27:01 GMT date: Thu, 17 Feb 2022 06:27:01 GMT < content-type: application/json content-type: application/json < content-length: 43 content-length: 43 < * Connection #0 to host prod2-lobby.triplea-game.org left intact {"code":404,"message":"HTTP 404 Not Found"}* Closing connection 0So the issue is not a code problem but a deployment one. One of two things is incorrect in the production environment:
A: The nginx web server does not have the latest config deployed to it to properly proxy requests to the correct lobby.
B: The 2.6 lobby as a whole, has not been deployed or is setup but running a 2.5 lobby.
Until the 2.6 lobby is deployed and the nginx web servers config is updated, all 2.6 client requests for maps are going to return 404.
This card I found on the Project schedule (https://github.com/triplea-game/triplea/projects/25#card-75012745) suggests that the 2.6 lobby has not been deployed but can be at any time. If this gets deployed, we can close 6 or more issues and be closer to releasing 2.6.
-
@magicstyck Nice work
yea I'm not aware of a 2.6 lobby yet. @RoiEX do you know if one is up -
Before you invest too much time, please test the download maps - issue using the current prerelease.
I was just able to download maps using 2.6.571.I don't know what might have fixed the issue - nobody told us. So I was surprised when it just worked.
Don't always trust TripleA when it comes to rules questions. Know the rules before you start … and better check what TripleA has done.
-
@ubernaut
Feel free to change it. It is not my list, but ours and everyone is welcomed to keep it updated. -
@LaFayette Can you explain to the group what the download maps issue is and why it was fixed with 2.6.571 according to @panther?
(see also the analysis made by @MagicStyck below)
-
Don't always trust TripleA when it comes to rules questions. Know the rules before you start … and better check what TripleA has done.
-
@frigoref said in TripleA development:
@panther can you please also label the issues you consider 2.6 blocking issues?
Do you need help for this?Is there anyone interested in assisting panter?
Yes, I'll do. At least those I am able to identify, such as those named above.
-
@panther Thanks for the link! If I understand LaFayette correctly, the "find the server" problem should be solved, but the database might be crashing sometimes. I offered to help with an issue there if I get access to it and he is about to make this possible.
However, he also told me that he made some adjustments so the database shouldn't be crashing anymore.
@LaFayette Please get in contact with me to explain on the access part and leave a comment here on what the currently remainig issue (for the download maps topic) is that needs solving. -
2.6.571 and the current 2.6.574 both give the same 404 error. That is because the client still defaults to prod2-lobby.triplea-game.org (https://github.com/triplea-game/triplea/pull/9682). When you change the Engine Setting to point at prod.triplea-game.org you get the TLS certificate error discussed in that PR and you are still unable to download the maps.
Reading through the issue here: https://forums.triplea-game.org/topic/3109/unable-to-download-maps-in-2-6-535/13 there are still issues with the production environment for 2.6. They are as follows:
- No valid certificate for prod.triplea-game.org
- database is crashing for prod.triplea-game.org - includes but not limited to disk space issue
The client 2.6 client gives you the link to pull down the maps manually to get around the 2.6 lobby server not working.
-
Of course I have switched the settings to Pre Release in Engine-Preferences / Testing before, as advised by @LaFayette .
To me my "improved" experience is that even when changing to Pre-Release the download maps feature did NOT work until my new test yesterday.
Just for completeness.
Hello! It looks like you're interested in this conversation, but you don't have an account yet.
Getting fed up of having to scroll through the same posts each visit? When you register for an account, you'll always come back to exactly where you were before, and choose to be notified of new replies (either via email, or push notification). You'll also be able to save bookmarks and upvote posts to show your appreciation to other community members.
With your input, this post could be even better 💗
Register Login