RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-10-31 17:52:45 +00:00
Code Issues Activity Actions

LibCore: Use URL in CHttpRequest

Browse source 
Now there's just CHttpRequest::set_url(URL), no need to specify the
host, port and path manually anymore.

Updated ChanViewer and Downloader for the API change.
This commit is contained in:
Andreas Kling 2019-08-10 19:32:03 +02:00
parent fb636389d6
commit ee83b1bcf4
6 changed files with 11 additions and 19 deletions
Download patch file Download diff file Expand all files Collapse all files

3
Applications/ChanViewer/BoardListModel.cpp
View file

@ -19,8 +19,7 @@ BoardListModel::~BoardListModel()
void BoardListModel::update()
{
CHttpRequest request;
request.set_hostname("a.4cdn.org");
request.set_path("/boards.json");
request.set_url("http://a.4cdn.org/boards.json");
auto* job = request.schedule();

3
Applications/ChanViewer/ThreadCatalogModel.cpp
View file

@ -27,8 +27,7 @@ void ThreadCatalogModel::set_board(const String& board)
void ThreadCatalogModel::update()
{
CHttpRequest request;
request.set_hostname("a.4cdn.org");
request.set_path(String::format("/%s/catalog.json", m_board.characters()));
request.set_url(String::format("http://a.4cdn.org/%s/catalog.json", m_board.characters()));
auto* job = request.schedule();

3
Applications/Downloader/main.cpp
View file

@ -9,8 +9,7 @@ int main(int argc, char** argv)
GApplication app(argc, argv);
CHttpRequest request;
request.set_hostname("www.google.com");
request.set_path("/");
request.set_url("http://www.google.com/");
auto job = request.schedule();
job->on_finish = [&job](bool success) {

2
Libraries/LibCore/CHttpJob.cpp
View file

@ -121,7 +121,7 @@ void CHttpJob::start()
dbg() << "CHttpJob: on_connected callback";
on_socket_connected();
};
bool success = m_socket->connect(m_request.hostname(), m_request.port());
bool success = m_socket->connect(m_request.url().host(), m_request.url().port());
if (!success)
return did_fail(CNetworkJob::Error::ConnectionFailed);
}

4
Libraries/LibCore/CHttpRequest.cpp
View file

@ -36,9 +36,9 @@ ByteBuffer CHttpRequest::to_raw_request() const
StringBuilder builder;
builder.append(method_name());
builder.append(' ');
builder.append(m_path);
builder.append(m_url.path());
builder.append(" HTTP/1.0\r\nHost: ");
builder.append(m_hostname);
builder.append(m_url.host());
builder.append("\r\n\r\n");
return builder.to_byte_buffer();
}

15
Libraries/LibCore/CHttpRequest.h
View file

@ -1,6 +1,7 @@
#pragma once
#include <AK/AKString.h>
#include <AK/URL.h>
class CNetworkJob;
@ -16,14 +17,10 @@ public:
CHttpRequest();
~CHttpRequest();
String hostname() const { return m_hostname; }
int port() const { return m_port; }
String path() const { return m_path; }
Method method() const { return m_method; }
const URL& url() const { return m_url; }
void set_url(const URL& url) { m_url = url; }
void set_hostname(const String& hostname) { m_hostname = hostname; }
void set_port(int port) { m_port = port; }
void set_path(const String& path) { m_path = path; }
Method method() const { return m_method; }
void set_method(Method method) { m_method = method; }
String method_name() const;
@ -32,8 +29,6 @@ public:
CNetworkJob* schedule();
private:
String m_hostname;
String m_path;
int m_port { 80 };
URL m_url;
Method m_method { GET };
};