Address review comments
This commit is contained in:
		
							
								
								
									
										2
									
								
								externals/CMakeLists.txt
									
									
									
									
										vendored
									
									
								
							
							
						
						
									
										2
									
								
								externals/CMakeLists.txt
									
									
									
									
										vendored
									
									
								
							| @@ -80,7 +80,7 @@ if (ENABLE_WEB_SERVICE) | ||||
|     set(LIBRESSL_SKIP_INSTALL ON CACHE BOOL "") | ||||
|     add_subdirectory(libressl EXCLUDE_FROM_ALL) | ||||
|     target_include_directories(ssl INTERFACE ./libressl/include) | ||||
| 	target_compile_definitions(ssl PRIVATE -DHAVE_INET_NTOP) | ||||
|     target_compile_definitions(ssl PRIVATE -DHAVE_INET_NTOP) | ||||
|  | ||||
|     # lurlparser | ||||
|     add_subdirectory(lurlparser EXCLUDE_FROM_ALL) | ||||
|   | ||||
| @@ -3,10 +3,10 @@ | ||||
| // Refer to the license.txt file included. | ||||
|  | ||||
| #include <future> | ||||
| #include <json.hpp> | ||||
| #include "common/detached_tasks.h" | ||||
| #include "common/logging/log.h" | ||||
| #include "web_service/announce_room_json.h" | ||||
| #include "web_service/json.h" | ||||
| #include "web_service/web_backend.h" | ||||
|  | ||||
| namespace AnnounceMultiplayerRoom { | ||||
|   | ||||
| @@ -15,10 +15,10 @@ namespace WebService { | ||||
|  | ||||
| constexpr std::array<const char, 1> API_VERSION{'1'}; | ||||
|  | ||||
| constexpr u32 HTTP_PORT = 80; | ||||
| constexpr u32 HTTPS_PORT = 443; | ||||
| constexpr int HTTP_PORT = 80; | ||||
| constexpr int HTTPS_PORT = 443; | ||||
|  | ||||
| constexpr u32 TIMEOUT_SECONDS = 30; | ||||
| constexpr std::size_t TIMEOUT_SECONDS = 30; | ||||
|  | ||||
| Client::JWTCache Client::jwt_cache{}; | ||||
|  | ||||
|   | ||||
		Reference in New Issue
	
	Block a user