svrooij / sonos2mqtt

:speaker: Sonos mqtt controller. Control your speakers from your mqtt server. mqtt-smarthome insprired.
https://sonos2mqtt.svrooij.io/
MIT License
76 stars 18 forks source link

Improperly escaped port colon in CurrentURI #120

Closed dgmltn closed 3 years ago

dgmltn commented 4 years ago

Here's a snippet from my node red configuration. This gets added to sonos with the server:port colon urlescaped. Here's the error message: Screenshot_20201020-171518

const uri = "x-rincon-mp3radio://cde:8000/stream";

node.send({
    "topic": "devices/sonos/set/office/adv-command",
    "payload": {
        "cmd": "AVTransportService.SetAVTransportURI",
        "val": {
            "InstanceID": 0,
            "CurrentURI": uri,
            "CurrentURIMetaData": {
                "Title": "My Stream",
                "AlbumArtUri": "http://myserver.com/albumart.png",
                "UpnpClass": "object.item.audioItem.audioBroadcast"
            }
        }
    }
});
node.send({
    "topic": "devices/sonos/set/office/play",
    "payload": ''
});
svrooij commented 4 years ago

See https://github.com/svrooij/node-sonos-ts/issues/62

svrooij commented 4 years ago

@all-contributors please add @dgmltn for bugs

allcontributors[bot] commented 4 years ago

@svrooij

I've put up a pull request to add @dgmltn! :tada:

github-actions[bot] commented 4 years ago

:tada: This issue has been resolved in version 3.1.1-beta.1 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

github-actions[bot] commented 3 years ago

:tada: This issue has been resolved in version 3.1.1 :tada:

The release is available on:

Your semantic-release bot :package::rocket: