Merge pull request #1010 from jjlin/admin-url

Avoid double-slashes in the admin URL
This commit is contained in:
Daniel García 2020-05-31 00:22:46 +02:00 committed by GitHub
commit b03698fadb
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 1 deletions

View File

@ -63,7 +63,9 @@ fn admin_path() -> String {
/// Used for `Location` response headers, which must specify an absolute URI /// Used for `Location` response headers, which must specify an absolute URI
/// (see https://tools.ietf.org/html/rfc2616#section-14.30). /// (see https://tools.ietf.org/html/rfc2616#section-14.30).
fn admin_url() -> String { fn admin_url() -> String {
format!("{}{}", CONFIG.domain(), ADMIN_PATH) // Don't use CONFIG.domain() directly, since the user may want to keep a
// trailing slash there, particularly when running under a subpath.
format!("{}{}{}", CONFIG.domain_origin(), CONFIG.domain_path(), ADMIN_PATH)
} }
#[get("/", rank = 2)] #[get("/", rank = 2)]