Skip to content

Commit

Permalink
Added set_extra_headers() to WebSocketServer
Browse files Browse the repository at this point in the history
  • Loading branch information
rohanrhu committed Apr 12, 2022
1 parent c896ba8 commit fd4341f
Show file tree
Hide file tree
Showing 7 changed files with 25 additions and 3 deletions.
7 changes: 7 additions & 0 deletions modules/websocket/doc_classes/WebSocketServer.xml
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,13 @@
If [code]false[/code] is passed instead (default), you must call [PacketPeer] functions ([code]put_packet[/code], [code]get_packet[/code], etc.), on the [WebSocketPeer] returned via [code]get_peer(id)[/code] to communicate with the peer with given [code]id[/code] (e.g. [code]get_peer(id).get_available_packet_count[/code]).
</description>
</method>
<method name="set_extra_headers">
<return type="void" />
<argument index="0" name="headers" type="PackedStringArray" default="PackedStringArray()" />
<description>
Sets additional headers to be sent to clients during the HTTP handshake.
</description>
</method>
<method name="stop">
<return type="void" />
<description>
Expand Down
3 changes: 3 additions & 0 deletions modules/websocket/emws_server.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,9 @@
#include "emws_server.h"
#include "core/os/os.h"

void EMWSServer::set_extra_headers(const Vector<String> &p_headers) {
}

Error EMWSServer::listen(int p_port, Vector<String> p_protocols, bool gd_mp_api) {
return FAILED;
}
Expand Down
1 change: 1 addition & 0 deletions modules/websocket/emws_server.h
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ class EMWSServer : public WebSocketServer {

public:
Error set_buffers(int p_in_buffer, int p_in_packets, int p_out_buffer, int p_out_packets) override;
void set_extra_headers(const Vector<String> &p_headers) override;
Error listen(int p_port, Vector<String> p_protocols = Vector<String>(), bool gd_mp_api = false) override;
void stop() override;
bool is_listening() const override;
Expand Down
1 change: 1 addition & 0 deletions modules/websocket/websocket_server.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ WebSocketServer::~WebSocketServer() {

void WebSocketServer::_bind_methods() {
ClassDB::bind_method(D_METHOD("is_listening"), &WebSocketServer::is_listening);
ClassDB::bind_method(D_METHOD("set_extra_headers", "headers"), &WebSocketServer::set_extra_headers, DEFVAL(Vector<String>()));
ClassDB::bind_method(D_METHOD("listen", "port", "protocols", "gd_mp_api"), &WebSocketServer::listen, DEFVAL(Vector<String>()), DEFVAL(false));
ClassDB::bind_method(D_METHOD("stop"), &WebSocketServer::stop);
ClassDB::bind_method(D_METHOD("has_peer", "id"), &WebSocketServer::has_peer);
Expand Down
1 change: 1 addition & 0 deletions modules/websocket/websocket_server.h
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,7 @@ class WebSocketServer : public WebSocketMultiplayerPeer {
uint32_t handshake_timeout = 3000;

public:
virtual void set_extra_headers(const Vector<String> &p_headers) = 0;
virtual Error listen(int p_port, const Vector<String> p_protocols = Vector<String>(), bool gd_mp_api = false) = 0;
virtual void stop() = 0;
virtual bool is_listening() const = 0;
Expand Down
11 changes: 9 additions & 2 deletions modules/websocket/wsl_server.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ bool WSLServer::PendingPeer::_parse_request(const Vector<String> p_protocols, St
return true;
}

Error WSLServer::PendingPeer::do_handshake(const Vector<String> p_protocols, uint64_t p_timeout, String &r_resource_name) {
Error WSLServer::PendingPeer::do_handshake(const Vector<String> p_protocols, uint64_t p_timeout, String &r_resource_name, const Vector<String> &p_extra_headers) {
if (OS::get_singleton()->get_ticks_msec() - time > p_timeout) {
print_verbose(vformat("WebSocket handshake timed out after %.3f seconds.", p_timeout * 0.001));
return ERR_TIMEOUT;
Expand Down Expand Up @@ -141,6 +141,9 @@ Error WSLServer::PendingPeer::do_handshake(const Vector<String> p_protocols, uin
if (!protocol.is_empty()) {
s += "Sec-WebSocket-Protocol: " + protocol + "\r\n";
}
for (int i = 0; i < p_extra_headers.size(); i++) {
s += p_extra_headers[i] + "\r\n";
}
s += "\r\n";
response = s.utf8();
has_request = true;
Expand All @@ -167,6 +170,10 @@ Error WSLServer::PendingPeer::do_handshake(const Vector<String> p_protocols, uin
return OK;
}

void WSLServer::set_extra_headers(const Vector<String> &p_headers) {
_extra_headers = p_headers;
}

Error WSLServer::listen(int p_port, const Vector<String> p_protocols, bool gd_mp_api) {
ERR_FAIL_COND_V(is_listening(), ERR_ALREADY_IN_USE);

Expand Down Expand Up @@ -199,7 +206,7 @@ void WSLServer::poll() {
for (const Ref<PendingPeer> &E : _pending) {
String resource_name;
Ref<PendingPeer> ppeer = E;
Error err = ppeer->do_handshake(_protocols, handshake_timeout, resource_name);
Error err = ppeer->do_handshake(_protocols, handshake_timeout, resource_name, _extra_headers);
if (err == ERR_BUSY) {
continue;
} else if (err != OK) {
Expand Down
4 changes: 3 additions & 1 deletion modules/websocket/wsl_server.h
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ class WSLServer : public WebSocketServer {
CharString response;
int response_sent = 0;

Error do_handshake(const Vector<String> p_protocols, uint64_t p_timeout, String &r_resource_name);
Error do_handshake(const Vector<String> p_protocols, uint64_t p_timeout, String &r_resource_name, const Vector<String> &p_extra_headers);
};

int _in_buf_size = DEF_BUF_SHIFT;
Expand All @@ -73,9 +73,11 @@ class WSLServer : public WebSocketServer {
List<Ref<PendingPeer>> _pending;
Ref<TCPServer> _server;
Vector<String> _protocols;
Vector<String> _extra_headers;

public:
Error set_buffers(int p_in_buffer, int p_in_packets, int p_out_buffer, int p_out_packets) override;
void set_extra_headers(const Vector<String> &p_headers) override;
Error listen(int p_port, const Vector<String> p_protocols = Vector<String>(), bool gd_mp_api = false) override;
void stop() override;
bool is_listening() const override;
Expand Down

0 comments on commit fd4341f

Please sign in to comment.