shayaantx / botdarr

Slack/Discord/Telegram/Matrix bot for accessing radarr, sonarr, and lidarr
GNU General Public License v3.0
157 stars 12 forks source link

Can't add movie, due to French title? #76

Closed Aerya closed 2 years ago

Aerya commented 2 years ago

Radarr is connected to the bot:

I can search for a movie

But can't add it

Nothing in the logs

shayaantx commented 2 years ago

@Aerya what version of radarr are you running?

is there anything in the radarr or network logs?

Did adding other non french named films work?

Aerya commented 2 years ago

Radarr:

Version
    4.2.0.6272
Version du package
    4.2.0.6272-ls304 by [linuxserver.io](https://www.linuxserver.io/)

Trace logs concerning this req

2022-05-22 17:23:41.1|Trace|Scheduler|Pending Tasks: 0
2022-05-22 17:23:49.1|Trace|Http|Req: 132399 [GET] /api/v3/movie?apikey=(removed) (from 172.*.*.188 GuzzleHttp/7)
2022-05-22 17:23:49.1|Trace|ConfigService|Using default config value for 'availabilitydelay' defaultValue:'0'
2022-05-22 17:23:49.4|Trace|Http|Res: 132399 [GET] /api/v3/movie?apikey=(removed) 200.OK (241 ms)
2022-05-22 17:23:49.4|Debug|Api|[GET] /api/v3/movie?apikey=(removed) 200.OK (241 ms)
2022-05-22 17:23:49.7|Trace|Http|Req: 132400 [GET] /api/v3/queue?apikey=(removed) (from 172.*.*.166 GuzzleHttp/7)
2022-05-22 17:23:49.7|Trace|Http|Res: 132400 [GET] /api/v3/queue?apikey=(removed) 200.OK (2 ms)
2022-05-22 17:23:49.7|Debug|Api|[GET] /api/v3/queue?apikey=(removed) 200.OK (2 ms)
2022-05-22 17:24:03.8|Trace|Http|Req: 132401 [GET] /api/v3/movie/lookup?apikey=(removed)&term=avatar+%3A+la+voie+de+l%27eau (from 10.0.4.94 Apache-HttpClient/4.5.13 (Java/1.8.0_332))
2022-05-22 17:24:03.8|Debug|Parser|Parsing string 'avatar : la voie de l'eau'
2022-05-22 17:24:03.8|Debug|Parser|Unable to parse avatar : la voie de l'eau
2022-05-22 17:24:03.8|Trace|HttpClient|Req: [GET] https://api.radarr.video/v1/search?q=avatar%2B%3A%2Bla%2Bvoie%2Bde%2Bl%27eau&year=
2022-05-22 17:24:04.1|Trace|HttpClient|Res: [GET] https://api.radarr.video/v1/search?q=avatar%2B%3A%2Bla%2Bvoie%2Bde%2Bl%27eau&year=: 200.OK (29265 bytes) (258 ms)
2022-05-22 17:24:04.1|Trace|ConfigService|Using default config value for 'certificationcountry' defaultValue:'US'
2022-05-22 17:24:04.1|Trace|ConfigService|Using default config value for 'availabilitydelay' defaultValue:'0'
2022-05-22 17:24:04.1|Trace|Http|Res: 132401 [GET] /api/v3/movie/lookup?apikey=(removed)&term=avatar+%3A+la+voie+de+l%27eau: 200.OK (274 ms)
2022-05-22 17:24:04.1|Debug|Api|[GET] /api/v3/movie/lookup?apikey=(removed)&term=avatar+%3A+la+voie+de+l%27eau: 200.OK (274 ms)
2022-05-22 17:24:04.2|Trace|Http|Req: 132402 [POST] /api/v3/movie?apikey=(removed) (from 10.0.4.94 Apache-HttpClient/4.5.13 (Java/1.8.0_332))
2022-05-22 17:24:04.2|Error|Radarr.Api.V3.Movies.MovieController.AddMovie (Radarr.Api.V3)|{
  "$.overview": {
    "errors": [
      {
        "errorMessage": "The JSON value could not be converted to System.String. Path: $.overview | LineNumber: 0 | BytePositionInLine: 788."
      }
    ],
    "validationState": "invalid",
    "isContainerNode": false
  }
}
2022-05-22 17:24:04.2|Trace|Http|Res: 132402 [POST] /api/v3/movie?apikey=(removed) 400.BadRequest (17 ms)
2022-05-22 17:24:04.2|Debug|Api|[POST] /api/v3/movie?apikey=(removed) 400.BadRequest (17 ms)
2022-05-22 17:24:10.5|Trace|Http|Req: 132403 [GET] /api/v3/movie?apikey=(removed) (from 162.*.*.132 GuzzleHttp/7)
2022-05-22 17:24:10.5|Trace|ConfigService|Using default config value for 'availabilitydelay' defaultValue:'0'
2022-05-22 17:24:10.8|Trace|Http|Res: 132403 [GET] /api/v3/movie?apikey=(removed) 200.OK (273 ms)
2022-05-22 17:24:10.8|Debug|Api|[GET] /api/v3/movie?apikey=(removed) 200.OK (273 ms)
2022-05-22 17:24:11.1|Trace|Http|Req: 132404 [GET] /api/v3/queue?apikey=(removed) (from 172.*.*.67 GuzzleHttp/7)
2022-05-22 17:24:11.1|Trace|Http|Res: 132404 [GET] /api/v3/queue?apikey=(removed) 200.OK (4 ms)
shayaantx commented 2 years ago

I'll investigate

Aerya commented 2 years ago

Same with English title

2022-05-22 17:29:55.0|Debug|Parser|Parsing string 'batman begins'
2022-05-22 17:29:55.0|Debug|Parser|Unable to parse batman begins
2022-05-22 17:29:55.4|Debug|Api|[GET] /api/v3/movie/lookup?apikey=(removed)&term=batman+begins: 200.OK (317 ms)
2022-05-22 17:29:55.4|Error|Radarr.Api.V3.Movies.MovieController.AddMovie (Radarr.Api.V3)|{
  "$.overview": {
    "errors": [
      {
        "errorMessage": "The JSON value could not be converted to System.String. Path: $.overview | LineNumber: 0 | BytePositionInLine: 745."
      }
    ],
    "validationState": "invalid",
    "isContainerNode": false
  }
}
2022-05-22 17:29:55.4|Debug|Api|[POST] /api/v3/movie?apikey=(removed) 400.BadRequest (9 ms)
2022-05-22 17:30:06.9|Debug|Parser|Parsing string 'batman begins'
2022-05-22 17:30:06.9|Debug|Parser|Unable to parse batman begins
2022-05-22 17:30:07.1|Debug|Api|[GET] /api/v3/movie/lookup?apikey=(removed)&term=batman+begins: 200.OK (144 ms)
2022-05-22 17:30:07.1|Error|Radarr.Api.V3.Movies.MovieController.AddMovie (Radarr.Api.V3)|{
  "$.overview": {
    "errors": [
      {
        "errorMessage": "The JSON value could not be converted to System.String. Path: $.overview | LineNumber: 0 | BytePositionInLine: 745."
      }
    ],
    "validationState": "invalid",
    "isContainerNode": false
  }
}
2022-05-22 17:30:07.1|Debug|Api|[POST] /api/v3/movie?apikey=(removed) 400.BadRequest (16 ms)
2022-05-22 17:30:11.2|Debug|SabnzbdProxy|Url: http://10.0.4.65:8077/api?mode=queue&start=0&limit=0&apikey=(removed)&output=json
2022-05-22 17:30:11.2|Debug|SabnzbdProxy|Url: http://10.0.4.65:8077/api?mode=history&start=0&limit=60&category=premovies&apikey=(removed)&output=json
2022-05-22 17:30:11.9|Debug|Api|[GET] /api/v3/queue/details: 200.OK (3 ms)
2022-05-22 17:30:11.9|Debug|Api|[GET] /api/v3/queue/details: 200.OK (5 ms)
shayaantx commented 2 years ago

@Aerya yea I just reproduced it, something to do with encoding for non english languages. I gtg for right now, but will investigate tonight, and push something that fixes (to develop and the slash discord branch)

Aerya commented 2 years ago

NP mate, no hurry :)

shayaantx commented 2 years ago

I was able to fix this here https://github.com/shayaantx/botdarr/pull/77, I need to test sonarr and lidarr later tonight, then I'll merge.

but it did work with the movie you tried

image

shayaantx commented 2 years ago

merged to 5.3.5 @Aerya , I did merge to the slash command feature branch, but somehow i broke test cases during the merge.

I will look at it later tonight or 2moro morning (still want to do https://github.com/shayaantx/botdarr/issues/75 before releasing slash commands + i need to write some unit tests around the slash commands)

shayaantx commented 2 years ago

fyi @Aerya pushed these changes to slash command docker tag

Aerya commented 2 years ago

I'm sorry I couldn't get back to you earlier. It works fine now, thank you!