search

axel3rd / mpg-coach-bot

MPG (Mon Petit Gazon) coach bot, to automate and optimize weekly actions
https://axel3rd.github.io/mpg-coach-bot/
MIT License
23 stars 8 forks source link

Updating team fails due to tactical composition (attackers & defenders switch) #91

Closed ghost closed 5 years ago

ghost commented 5 years ago

Describe the bug

Exception in thread "main" java.lang.UnsupportedOperationException: Unsupported status code: 403 Forbidden / Content: {"success":false,"error":"goodTry","code":839} at org.blondin.mpg.AbstractClient.call(AbstractClient.java:124) at org.blondin.mpg.AbstractClient.post(AbstractClient.java:85) at org.blondin.mpg.AbstractClient.post(AbstractClient.java:81) at org.blondin.mpg.root.MpgClient.updateCoach(MpgClient.java:95) at org.blondin.mpg.Main.updateTeamWithRetry(Main.java:190) at org.blondin.mpg.Main.processGames(Main.java:166) at org.blondin.mpg.Main.processLeague(Main.java:90) at org.blondin.mpg.Main.process(Main.java:68) at org.blondin.mpg.Main.main(Main.java:59)

Join debug files

Depending your championship, join data files set in attachment.

  1. The MPG JSon Response of these Request on https://api.monpetitgazon.com (visible via F12 in your browser):

    • From home : GET /user/dashboard
    • From coach: GET /league/[yourLeagueId]/coach
    • From transfer : GET /league/[yourLeagueId]/transfer/buy

  2. The Players statistics data, one JSon from:

  3. The Players statistics time update datas : https://www.mpgstats.fr/json/leagues.json

  4. The Injury / Suspended data, one full HTML from:

Expected behavior

A clear and concise description of what you expected to happen.

If problem on update team feature, please join the Request and Response (visible via F12 in your browser) of POST /league/[yourLeagueId]/coach when you save your team in MPG.

axel3rd commented 5 years ago

Hello @nicobal,

Is your league started ? Do you have the capacity to configure your team for the next match in https://mpg.football/league/[YourLeagueUid]/coach ?

Best regards

ghost commented 5 years ago

Hello @axel3rd , Yes I do. I would like to help you doing this "If problem on update team feature, please join the Request and Response (visible via F12 in your browser) of POST /league/[yourLeagueId]/coach when you save your team in MPG." but I don`t know how. I press F12. Could you please let me know where to find the Request and Response? Sorry for being a noob... ;-)

axel3rd commented 5 years ago

Could you please let me know where to find the Request and Response?

@nicobal : I had the Get MPG data for opening a bug wiki page for that, I hope it will help.

ghost commented 5 years ago

DASHBOARD {data: {,…}} data: {,…} follow: [] leagues: [{id: "LH9Y5CAY", admin_mpg_user_id: "mpg_user_306874", current_mpg_user: "mpg_user_306874",…},…] 0: {id: "LH9Y5CAY", admin_mpg_user_id: "mpg_user_306874", current_mpg_user: "mpg_user_306874",…} admin_mpg_user_id: "mpg_user_306874" championship: 5 current_mpg_user: "mpg_user_306874" id: "LH9Y5CAY" leagueStatus: 1 mode: 1 name: "Espresso Yourself League SP" teamStatus: null url: "assets/public/img/league/1.jpg" 1: {id: "qyOG7BuuZcv", admin_mpg_user_id: "mpg_user_306874", current_mpg_user: "mpg_user_306874",…} admin_mpg_user_id: "mpg_user_306874" championship: 1 current_mpg_user: "mpg_user_306874" id: "qyOG7BuuZcv" leagueStatus: 3 mode: 1 name: "Liga 1 Casas Bahia" players: 10 teamStatus: 1 url: "assets/public/img/league/7.jpg" 2: {id: "KJKVFJSW", admin_mpg_user_id: "mpg_user_306874", current_mpg_user: "mpg_user_306874",…} admin_mpg_user_id: "mpg_user_306874" championship: 1 current_mpg_user: "mpg_user_306874" id: "KJKVFJSW" leagueStatus: 3 mode: 1 name: "Plus Belge La Vie" players: 10 teamStatus: 1 url: "assets/public/img/league/4.jpg" 3: {id: "KJVB6L7C", admin_mpg_user_id: "mpg_user_352212", current_mpg_user: "mpg_user_306874",…} admin_mpg_user_id: "mpg_user_352212" championship: 2 current_mpg_user: "mpg_user_306874" id: "KJVB6L7C" leagueStatus: 4 mode: 1 name: "World Cup 2018 Part deux" players: 10 teamStatus: 2 url: "assets/public/img/league/6.jpg"

ghost commented 5 years ago

COACH {data: {realday: 1, dateMatch: 1565377200000, timetogame: 257975837, champid: 2,…}} data: {realday: 1, dateMatch: 1565377200000, timetogame: 257975837, champid: 2,…} bonus: {1: 1, 2: 1, 3: 2, 4: 3, 5: 1, 6: 1, 7: 1} bonusSelected: {} champid: 2 composition: 532 dateMatch: 1565377200000 fridayBrief: true matchId: "mpg_match_KJVB6L7C_3_1_1" money: 50 nanardpaid: 0 nbPlayers: 10 players: [{firstname: "Gerard", lastname: "Deulofeu", playerid: "player_94924", teamid: "57", position: 4,…},…] playersOnPitch: {1: "player_21205", 2: "player_203368", 3: "player_56192", 4: "player_39487", 5: "player_54484",…} realday: 1 stadium: "At The Pyramids" tacticalsubstitutes: [] tds: {update: "2019-07-15T13:06:56.421Z"} teamAway: {id: "mpg_team_KJVB6L7C$$mpg_user_1450718", userId: "mpg_user_1450718", name: "Dukes of Hazard",…} teamHome: {id: "mpg_team_KJVB6L7C$$mpg_user_306874", userId: "mpg_user_306874", name: "Salah Pas Fait",…} teams: {1: {name: "Man. United", status: "home", against: "Chelsea"},…} timetogame: 257975837

ghost commented 5 years ago

POST {success: "teamSaved"} success: "teamSaved"

ghost commented 5 years ago

Hope it helps!

axel3rd commented 5 years ago

@nicobal : Unfortunately, this content is not exploitable 😢. You seems have copy/paste the "browser view" result, with some "player_54484",…} => all data are not displayed. In FireFox, you should scroll at the bottom of "Response" tab to have the brut content (not modified/interpeted by the browser).

In addition, for POST /league/[yourLeagueId]/coach, the required data are the one in parameters tab of request (aka: The data which are sent to MPG), not the Reponse tab (the data which are reveived for this request).

ghost commented 5 years ago

sorry! will look into it is this better? DASHBOARD

{"data":{"leagues":[{"id":"LH9Y5CAY","admin_mpg_user_id":"mpg_user_306874","current_mpg_user":"mpg_user_306874","name":"Espresso Yourself League SP","url":"assets/public/img/league/1.jpg","leagueStatus":1,"championship":5,"mode":1,"teamStatus":null},{"id":"qyOG7BuuZcv","admin_mpg_user_id":"mpg_user_306874","current_mpg_user":"mpg_user_306874","name":"Liga 1 Casas Bahia","url":"assets/public/img/league/7.jpg","leagueStatus":3,"championship":1,"mode":1,"teamStatus":1,"players":10},{"id":"KJKVFJSW","admin_mpg_user_id":"mpg_user_306874","current_mpg_user":"mpg_user_306874","name":"Plus Belge La Vie","url":"assets/public/img/league/4.jpg","leagueStatus":3,"championship":1,"mode":1,"teamStatus":1,"players":10},{"id":"KJVB6L7C","admin_mpg_user_id":"mpg_user_352212","current_mpg_user":"mpg_user_306874","name":"World Cup 2018 Part deux","url":"assets/public/img/league/6.jpg","leagueStatus":4,"championship":2,"mode":1,"teamStatus":2,"players":10}],"follow":[]}}
ghost commented 5 years ago

COACH

{"data":{"realday":1,"dateMatch":1565377200000,"timetogame":201650808,"champid":2,"matchId":"mpg_match_KJVB6L7C_3_1_1","stadium":"At The Pyramids","teamHome":{"id":"mpg_team_KJVB6L7C$$mpg_user_306874","userId":"mpg_user_306874","name":"Salah Pas Fait","abbr":"SPF","coach":"Nico","jersey":{"id":2,"zones":{"z6":"#FFFFFF","z2":"#ff2626","z3":"#FFFFFF","z4":"#ff2626","z5":"#FFFFFF","z1":"#ff2626","z7":"#ff2626"},"sponsor":3},"jerseyHome":"jersey_backup_2_ed9bb436b2cd08fc7953a80408970e77","jerseyAway":"jersey_0_1561646533972","jerseyUrl":"https://s3-eu-west-3.amazonaws.com/jersey.mpg.football/prod/jersey_backup_2_ed9bb436b2cd08fc7953a80408970e77.png"},"teamAway":{"id":"mpg_team_KJVB6L7C$$mpg_user_1450718","userId":"mpg_user_1450718","name":"Dukes of Hazard","abbr":"DOH","coach":"Sid","jersey":{"id":0,"sponsor":10,"zones":{"z1":"#ff2626"}},"jerseyHome":"jersey_backup_0_8728634eb58bcbfabe2bffc2f68c44f5","jerseyAway":"jersey_backup_0_8728634eb58bcbfabe2bffc2f68c44f5","jerseyUrl":"https://s3-eu-west-3.amazonaws.com/jersey.mpg.football/prod/jersey_backup_0_8728634eb58bcbfabe2bffc2f68c44f5.png"},"nbPlayers":10,"players":[{"firstname":"Gerard","lastname":"Deulofeu","playerid":"player_94924","teamid":"57","position":4,"quotation":18,"ultraPosition":40,"lastFiveRate":{}},{"firstname":"Mateo","lastname":"Kovacic","playerid":"player_91651","teamid":"8","position":3,"quotation":13,"ultraPosition":32,"lastFiveRate":{}},{"firstname":null,"lastname":"Jota","playerid":"player_89274","teamid":"7","position":3,"quotation":7,"ultraPosition":32,"lastFiveRate":{}},{"firstname":"Josip","lastname":"Drmic","playerid":"player_81048","teamid":"45","position":4,"quotation":8,"ultraPosition":40,"lastFiveRate":{}},{"firstname":"Olivier","lastname":"Giroud","playerid":"player_44346","teamid":"8","position":4,"quotation":10,"ultraPosition":40,"lastFiveRate":{}},{"firstname":"Moussa","lastname":"Djenepo","playerid":"player_431131","teamid":"20","position":3,"quotation":12,"ultraPosition":32,"lastFiveRate":{}},{"firstname":"Erik","lastname":"Pieters","playerid":"player_39487","teamid":"90","position":2,"quotation":13,"ultraPosition":21,"lastFiveRate":{}},{"firstname":null,"lastname":"Rui Patrício","playerid":"player_38533","teamid":"39","position":1,"quotation":15,"ultraPosition":10,"lastFiveRate":{}},{"firstname":"Lukasz","lastname":"Fabianski","playerid":"player_37096","teamid":"21","position":1,"quotation":25,"ultraPosition":10,"lastFiveRate":{}},{"firstname":"Tanguy","lastname":"Ndombele","playerid":"player_231372","teamid":"6","position":3,"quotation":22,"ultraPosition":31,"lastFiveRate":{}},{"firstname":"Issa","lastname":"Diop","playerid":"player_219924","teamid":"21","position":2,"quotation":21,"ultraPosition":20,"lastFiveRate":{}},{"firstname":"Pablo","lastname":"Fornals","playerid":"player_217593","teamid":"21","position":3,"quotation":13,"ultraPosition":32,"lastFiveRate":{}},{"firstname":null,"lastname":"Wesley","playerid":"player_213345","teamid":"7","position":4,"quotation":12,"ultraPosition":40,"lastFiveRate":{}},{"firstname":"Tom","lastname":"Heaton","playerid":"player_21205","teamid":"90","position":1,"quotation":18,"ultraPosition":10,"lastFiveRate":{}},{"firstname":"Oleksandr","lastname":"Zinchenko","playerid":"player_206325","teamid":"43","position":3,"quotation":10,"ultraPosition":31,"lastFiveRate":{}},{"firstname":"Frederic","lastname":"Guilbert","playerid":"player_203368","teamid":"7","position":2,"quotation":13,"ultraPosition":21,"lastFiveRate":{}},{"firstname":"Daniel","lastname":"James","playerid":"player_200617","teamid":"1","position":3,"quotation":1,"ultraPosition":32,"lastFiveRate":{}},{"firstname":"Anwar","lastname":"El Ghazi","playerid":"player_193488","teamid":"7","position":4,"quotation":9,"ultraPosition":40,"lastFiveRate":{}},{"firstname":"Mason","lastname":"Mount","playerid":"player_184341","teamid":"8","position":3,"quotation":10,"ultraPosition":32,"lastFiveRate":{}},{"firstname":null,"lastname":"Angeliño","playerid":"player_145235","teamid":"43","position":2,"quotation":9,"ultraPosition":21,"lastFiveRate":{}},{"firstname":"Leandro","lastname":"Trossard","playerid":"player_116216","teamid":"36","position":3,"quotation":11,"ultraPosition":32,"lastFiveRate":{}},{"firstname":"Raúl","lastname":"Jiménez","playerid":"player_102057","teamid":"39","position":4,"quotation":28,"ultraPosition":40,"lastFiveRate":{}},{"firstname":"Jannik","lastname":"Vestergaard","playerid":"player_93100","teamid":"20","position":2,"quotation":12,"ultraPosition":20,"lastFiveRate":{}},{"firstname":null,"lastname":"Cédric Soares","playerid":"player_58822","teamid":"20","position":2,"quotation":9,"ultraPosition":21,"lastFiveRate":{}},{"firstname":null,"lastname":"Cuco Martina","playerid":"player_56192","teamid":"11","position":2,"quotation":12,"ultraPosition":21,"lastFiveRate":{}},{"firstname":null,"lastname":"Kiko Femenía","playerid":"player_54484","teamid":"57","position":2,"quotation":11,"ultraPosition":21,"lastFiveRate":{}},{"firstname":"José","lastname":"Holebas","playerid":"player_40868","teamid":"57","position":2,"quotation":10,"ultraPosition":21,"lastFiveRate":{}},{"firstname":"Björn","lastname":"Engels","playerid":"player_108156","teamid":"7","position":2,"quotation":10,"ultraPosition":20,"lastFiveRate":{}},{"firstname":"Craig","lastname":"Dawson","playerid":"player_60232","teamid":"57","position":2,"quotation":6,"ultraPosition":20,"lastFiveRate":{}},{"firstname":"Richard","lastname":"Stearman","playerid":"player_18832","teamid":"49","position":2,"quotation":12,"ultraPosition":20,"lastFiveRate":{}}],"bonus":{"1":1,"2":1,"3":2,"4":3,"5":1,"6":1,"7":1},"nanardpaid":0,"money":50,"fridayBrief":true,"tacticalsubstitutes":[],"composition":532,"playersOnPitch":{"1":"player_21205","2":"player_203368","3":"player_56192","4":"player_39487","5":"player_54484","6":"player_40868","7":"player_431131","8":"player_217593","9":"player_91651","10":"player_102057","11":"player_94924","12":null,"13":null,"14":null,"15":null,"16":null,"17":null,"18":null},"bonusSelected":{},"teams":{"1":{"name":"Man. United","status":"home","against":"Chelsea"},"3":{"name":"Arsenal","status":"away","against":"Newcastle"},"4":{"name":"Newcastle","status":"home","against":"Arsenal"},"6":{"name":"Tottenham","status":"home","against":"Aston Villa"},"7":{"name":"Aston Villa","status":"away","against":"Tottenham"},"8":{"name":"Chelsea","status":"away","against":"Man. United"},"11":{"name":"Everton","status":"away","against":"Crystal Palace"},"13":{"name":"Leicester","status":"home","against":"Wolverhampton"},"14":{"name":"Liverpool","status":"home","against":"Norwich"},"20":{"name":"Southampton","status":"away","against":"Burnley"},"21":{"name":"West Ham","status":"home","against":"Man. City"},"31":{"name":"Crystal Palace","status":"home","against":"Everton"},"36":{"name":"Brighton","status":"away","against":"Watford"},"39":{"name":"Wolverhampton","status":"away","against":"Leicester"},"43":{"name":"Man. City","status":"away","against":"West Ham"},"45":{"name":"Norwich","status":"away","against":"Liverpool"},"49":{"name":"Sheffield","status":"away","against":"Bournemouth"},"57":{"name":"Watford","status":"home","against":"Brighton"},"90":{"name":"Burnley","status":"home","against":"Southampton"},"91":{"name":"Bournemouth","status":"home","against":"Sheffield"}},"tds":{"update":"2019-07-15T13:06:56.421Z"}}}
ghost commented 5 years ago

POST `

bonusSelected {}
composition 532
matchId mpg_match_KJVB6L7C_3_1_1
playersOnPitch {…}
1 player_21205
2 player_203368
3 player_56192
4 player_39487
5 player_54484
6 player_40868
7 player_431131
8 player_217593
9 player_91651
10 player_102057
11 player_94924
12 null
13 null
14 null
15 null
16 null
17 null
18 null
realday 1
tacticalsubstitutes []

`

axel3rd commented 5 years ago

@nicobal : For GET /user/dashboard & GET /league/[yourLeagueId]/coach, content is relevant. For POST /league/[yourLeagueId]/coach (team update - your previous comment), no ; the content should be a "one line" data like { "playersOnPitch": [content-deleted] }.

Checking your dahsboard request, you have 4 leaques. The content of coach is about KJVB6L7C league (England championship). Can you confirm this is the team update which is failing via mpg-caoch-boat ?

axel3rd commented 5 years ago

@nicobal : Perhaps a more simple way for fixing this team update problem :

... It is not an elegant solution, but could reduce the number of interactions to achieve a fix.

axel3rd commented 5 years ago

Root cause found : on a 5-3-2 tactical composition, the boot try to send a 2-5-3 :confounded: