beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-22 07:30:19 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

URL: No need to include ":80" when serializing http:// URLs

Browse source
This commit is contained in:
Andreas Kling 2019-10-10 22:06:25 +02:00
parent c294c97cdf
commit bf5a65d934
Notes: sideshowbarker 2024-07-19 11:44:15 +09:00 
Author: https://github.com/awesomekling
Commit: https://github.com/SerenityOS/serenity/commit/bf5a65d934a
1 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
AK/URL.cpp
View file

@ -125,8 +125,10 @@ String URL::to_string() const
builder.append("://");
if (protocol() != "file") {
builder.append(m_host);
builder.append(':');
builder.append(String::number(m_port));
if (protocol() != "http" || port() != 80) {
builder.append(':');
builder.append(String::number(m_port));
}
}
builder.append(m_path);
return builder.to_string();