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

Ombi.Api.Lidarr: Remove unused fields from ArtistAdd #4727

Merged

Commits on Aug 11, 2022

  1. Ombi.Api.Lidarr: Remove unused fields from ArtistAdd

    When an artist is not found in Lidarr as part of requesting an album,
    MusicSender will make a POST request against the /api/v1/artist endpoint
    to add such artist.
    
    Not all fields defined in ArtistAdd are initialized, and those
    uninitialized will be `null` values in the JSON body of the request, as
    shown in this intercepted request:
    
    ```
    {
        "addOptions": {
            "AlbumsToMonitor": [
                "e5c48b66-44ef-3685-ad53-45dbcd7294c0"
            ],
            "monitor": 6,
            "monitored": true,
            "searchForMissingAlbums": false
        },
        "added": "2022-08-10T06:49:32.4374278+00:00",
        "albumFolder": true,
        "artistName": "Manolo García",
        "cleanName": "manologarcía",
        "disambiguation": null,
        "discogsId": 0,
        "ended": false,
        "foreignArtistId": "1c8309da-9789-40bf-b9c2-e20064263820",
        "images": [],
        "links": [],
        "metadataProfileId": 1,
        "monitored": true,
        "overview": null,
        "qualityProfileId": 3,
        "ratings": null,
        "remotePoster": null,
        "rootFolderPath": "/media/music/",
        "sortName": null,
        "statistics": null,
        "status": null,
        "tadbId": 0,
        "tags": null
    }
    ```
    
    This request will fail and Lidarr will return a 400 BadRequest error
    with the following message:
    
    ```
    2022-08-10 01:45:52.458 +00:00 [Error] StatusCode: BadRequest, Reason: Bad Request, RequestUri: http://lidarr:8686/api/v1/artist
    2022-08-10 01:45:52.459 +00:00 [Debug] {
      "type": "https://tools.ietf.org/html/rfc7231#section-6.5.1",
      "title": "One or more validation errors occurred.",
      "status": 400,
      "traceId": "00-52e01b322a05d7c6633eca2488ef2a5c-06345b3bb8c4bb6c-00",
      "errors": {
        "$.status": [
          "The JSON value could not be converted to NzbDrone.Core.Music.ArtistStatusType. Path: $.status | LineNumber: 0 | BytePositionInLine: 14."
        ]
      }
    }
    ```
    
    Removing all the `null` fields from the JSON body fixes the problem and
    correctly adds the artist to Lidarr.
    mvicomoya committed Aug 11, 2022
    Configuration menu
    Copy the full SHA
    313a4b0 View commit details
    Browse the repository at this point in the history