Skip to content

Commit

Permalink
#107 avoid double slash in url
Browse files Browse the repository at this point in the history
  • Loading branch information
totaam committed Oct 22, 2021
1 parent d772a0a commit c1b5898
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion html5/connect.html
Original file line number Diff line number Diff line change
Expand Up @@ -686,8 +686,10 @@ <h4 class="panel-title">Advanced options</h4>
url += "/";
}
url += path;
if (!path.endsWith("/")) {
url += "/";
}
}
url += "/";
return url;
}

Expand Down

3 comments on commit c1b5898

@JanCVanB
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@totaam This allows host addresses that end with a port number and no slash suffix (https://abc.xyz:8080). Will that be a problem?

@totaam
Copy link
Collaborator Author

@totaam totaam commented on c1b5898 Oct 23, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will that be a problem?

@JanCVanB it would be, but I had already spotted it: a5af520

@JanCVanB
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@totaam 👍🏻

Please sign in to comment.