Add "Client-Name" to ws handshake headers

lavacord / Lavacord

A easy to use Lavalink client that can be used with any discord api wrapper

Apache License 2.0

78 stars 21 forks source link

Add "Client-Name" to ws handshake headers #112

Closed TristanWasTaken closed 1 year ago

TristanWasTaken commented 2 years ago

Add the required "Client-Name" header for ws connections as stated in IMPLEMENTATION.md

PapiOphidian commented 2 years ago

The header isn't required. The only thing it does if you don't supply it is log that lib devs should add it, but definitely 👍

TristanWasTaken commented 2 years ago

I'm still lingering around on older Lavalink versions and didn't really try 3.4 yet 😄 But yeah, the header isn't "required" for it to work, but it should be still added because it is specified in the implementation.