Как правильно записать ListenAndServeTLS() с кастомным типом servemux?

Question

[Vadim Rublev]

Ругается, что много аргументов в ListenAndServeTLS().

Как записать правильно? И почему всего два аргумента в этом случае (в документации-синтаксисе для ListenAndServeTLS() - 4)?
Код:

func main() {
    var mux = http.NewServeMux()
    mux.HandleFunc("/", indexPage)
    var serv = &http.Server {
        Addr:         serverPort,
        ReadTimeout:  15 * time.Second,
        WriteTimeout: 15 * time.Second,
    }
    log.Fatal(serv.ListenAndServeTLS(TLScert, TLSkey, mux))
}

Answer 1

[Евгений Мамонов]

Вы вызываете метод ListenAndServeTLS для структуры Server, у него всего два параметра.
Четыре параметра у метода ListenAndServeTLS для пакета http.

Вам нужно добавить параметр Handler: mux в структуру, и вызвать метод ListenAndServeTLS структуры Server. У вас должно получиться примерно так:

func main() {
    var mux = http.NewServeMux()
    mux.HandleFunc("/", indexPage)
    var serv = &http.Server{
        // добавляете параметр
        Handler:      mux,
        Addr:         serverPort,
        ReadTimeout:  15 * time.Second,
        WriteTimeout: 15 * time.Second,
    }
    // вызываете с двумя параметрами
    log.Fatal(serv.ListenAndServeTLS(TLScert, TLSkey))
}

Comments

[Евгений Мамонов]

получилось?

[Vadim Rublev]

Ошибки нет больше. То есть получилось.
Хотя маршрут не работает корректно. ) Но это уже другой вопрос, видимо... )

[Vadim Rublev]

Спасибо!

[Vadim Rublev]

...Теперь этот mux надо везде писать, что ли?
Маршрут молчит наглухо! Не пишет ошибок (кроме "неизвестный сертификат" на мой самоподписанный TLS) и в браузер ничего не выдает.

func indexPage(w http.ResponseWriter, r *http.Request) {
	http.FileServer(http.Dir(DirS))
}

[Евгений Мамонов]

C http.FileServer могут быть проблемы с путями и т.д.
Для теста лучше сделайте обработчик по проще, что-то типа такого

func indexPage(w http.ResponseWriter, r *http.Request) {
    w.WriteHeader(http.StatusOK)
    fmt.Fprintf(w, "OK")
}

"Теперь этот mux надо везде писать, что ли?" тут не совсем понятно о чём речь

[Vadim Rublev]

> "Теперь этот mux надо везде писать, что ли?" тут не совсем понятно о чём речь
Имел ввиду - надо писать mux вместо http.

Так у меня не тест. Рабочий сервер же.
http.FileServer(http.Dir(DirS)) открывает index-файл (который должен открываться по маршруту "/", вестимо).

[Евгений Мамонов]

"Имел ввиду - надо писать mux вместо http." - тут трудно сказать, зависит от того, что вам нужно сделать.

"Маршрут молчит наглухо! Не пишет ошибок (кроме "неизвестный сертификат" на мой самоподписанный TLS) и в браузер ничего не выдает" - а принудительно открыть в браузере не получается?

Если цель сделать чтобы заработало "http.FileServer(http.Dir(DirS))" - вот полностью рабочий вариант

1. Генерируем ключи, Common Name - ваш домен или localhost

req -x509 -nodes -newkey rsa:2048 -keyout server.key -out server.crt -days 3650

2. В папке с ключами создаём main.go с таким содержанием

package main

import (
    "log"
    "net/http"
    "time"
)

var listenAddr = ":9999"
var TLScert = "server.crt"
var TLSkey = "server.key"
var DirS = "/etc"

func main() {
    var mux = http.NewServeMux()

    mux.Handle("/", http.FileServer(http.Dir(DirS)))

    var serv = &http.Server{
        Addr:         listenAddr,
        Handler:      mux,
        ReadTimeout:  15 * time.Second,
        WriteTimeout: 15 * time.Second,
    }

    log.Fatal(serv.ListenAndServeTLS(TLScert, TLSkey))
}

3. Делаем запрос и проверяем ответ
curl -vvv -k https://localhost:9999/
-k заставляет curl не ругаться "unknown certificate authority / SEC_ERROR_UNTRUSTED_ISSUER"

4. Видим ответ типа

* Connected to localhost (127.0.0.1) port 9999 (#0)
* Initializing NSS with certpath: sql:/etc/pki/nssdb
* skipping SSL peer certificate verification
* SSL connection using TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256
* Server certificate:
*       subject: E=[e-mail],CN=localhost,O=Internet Widgits Pty Ltd,ST=Some-State,C=RU
*       start date: Jun 01 10:10:53 2021 GMT
*       expire date: May 30 10:10:53 2031 GMT
*       common name: localhost
*       issuer: E=[e-mail],CN=localhost,O=Internet Widgits Pty Ltd,ST=Some-State,C=RU
> GET / HTTP/1.1
> User-Agent: curl/7.29.0
> Host: localhost:9999
> Accept: */*
>
< HTTP/1.1 200 OK
< Content-Type: text/html; charset=utf-8
< Last-Modified: Sun, 23 May 2021 19:48:51 GMT
< Date: Tue, 01 Jun 2021 10:23:20 GMT
< Transfer-Encoding: chunked
<
<pre>
....
<a href="aliases">aliases</a>
<a href="aliases.db">aliases.db</a>
<a href="GeoIP.conf">GeoIP.conf</a>
....

[Евгений Мамонов]

По хорошему надо бы создать для этого новый вопрос, чтобы другие люди, когда столкнутся с подобной задачей могли найти ответ :)
Создадите?
Если что-то не получится - я буду рад помочь :)

[Vadim Rublev]

Тестирую...

Чтобы создать новый вопрос, его надо сформулировать. Чтобы сформулировать, надо четко понимать что происходит и что в итоге это дает... Попробую разобраться. (Гугл нифига про это не знает.)

[Vadim Rublev]

> а принудительно открыть в браузере не получается?
В смысле?..
Открыть HTML-файл в браузере получается. Но сервер тут ни причем.
Открыть принудительно_сервером файл - это же как раз это выражение: http.FileServer(http.Dir(DirS))

[Евгений Мамонов]

А что тогда не получается?
Попробуйте мой вариант, там серверная часть работает 100%, curl отдаёт результат.
Если у вас есть домен с реальным IP - можно сделать бесплатный SSL сертификат от Letsencrypt.

[Vadim Rublev]

Ваш пример рабочий, да. Отличается он функцией mux.Handle(), в моем коде это mux.HandleFunc(). То есть дело было не в FileServer().
В чем разница между Handle() и HandleFunc() так я и не догоняю. (Если смотреть тупо примеры всякие - в казалось бы одинаковых ситуациях в разных примерах бывает и Handle() и HandleFunc().) Ну да ладно.

[Vadim Rublev]

Работает в модели "все в куче" - вот как в вашем примере: обработчики вместе с сервером в функции main().
Но, не работает если каждый обработчик делать отдельной функцией [вне main()], а в main() только сервер и перечисление маршрутов с запуском функции-обработчика:

func indexPage(w http.ResponseWriter, r *http.Request) {
	http.FileServer(http.Dir(DirS))
}
func main() {
    var mux = http.NewServeMux()
    mux.Handle("/", indexPage)
    var serv = &http.Server{
        Handler:      mux,
        Addr:         serverPort,
        ReadTimeout:  15 * time.Second,
        WriteTimeout: 15 * time.Second,
    }
    log.Fatal(serv.ListenAndServeTLS(TLScert, TLSkey))
}

Как правильно запустить функцию indexPage()? Как ни пытался - ругается. (

[Vadim Rublev]

Домена с IP нет пока. Работаю на локальном сервере с самоподписанным-липовым сертификатом. )

[Евгений Мамонов]

> В чем разница между Handle() и HandleFunc() так я и не догоняю. (Если смотреть тупо примеры всякие - в казалось бы одинаковых ситуациях в разных примерах бывает и Handle() и HandleFunc().) Ну да ладно.

Как раз в этом и причина почему ваш пример не работает.
Разница в том, что обработчик с типом HandleFunc должен соответствовать типу "func(w http.ResponseWriter, r *http.Request)"
А с типом Handler - должен реализовывать интерфейс http.Handler.

type Handler interface {
    ServeHTTP(ResponseWriter, *Request)
}

Т.е. чтобы заработал именно ваш пример c mux.HandleFunc("/", indexPage) - реализация indexPage должна быть вот такой, (нужно вызвать метод ServeHTTP) тогда оно заработает.

func indexPage(w http.ResponseWriter, r *http.Request) {
  http.FileServer(http.Dir(DirS)).ServeHTTP(w, r)
}

Но такая реализация не оптимальна, т.к. приводит к лишнему вызову функции indexPage, вместо прямого вызова метода ServeHTTP, как следствие появляются лишние тормоза за счёт проброски данных через стек.

А если копнуть немного глубже и посмотреть реализацию функции http.FileServer

func FileServer(root FileSystem) Handler {
	return &fileHandler{root}
}

то мы увидим, что помимо лишнего вызова функции у нас еще и каждый раз будет создаваться структура fileHandler под которую каждый раз будет выделяться память, а потом освобождаться при помощи сборщика мусора, т.к. возвращается указатель, т.е. ваш вариант реализации вызовет приличную доп. нагрузку без видимых оснований для этого.

По этому лучше используйте реализацию из моего примера, там память под структуру fileHandler выделяется всего один раз, во время запуска.

[Евгений Мамонов]

> Работает в модели "все в куче" - вот как в вашем примере: обработчики вместе с сервером в функции main()
Вы хотите чтобы indexPage возвращал только index.html?

[Евгений Мамонов]

Еще момент, который вы могли упустить, под "/" в mux.Handle("/", indexPage) попадут всё, что угодно, т.е.
и https://localhost:9999/ , и https://localhost:9999/index.html, и https://localhost:9999/unknown_page
Т.е. все эти URL'ы под него попадут.

Напишите какая у вас задача, без понимание этого вам не просто помочь :)

[Vadim Rublev]

> Разница в том, что...
Ясно, что это разные функции, и технически они по-разному работают.
Но это никак не объясняет логику: для каких случаев та и другая функция; почему в конкретном месте то или иное применяется.
Я понимаю, что для тру-программиста (типа если спросить автора ЯП) это глупый вопрос - ибо в его техническом ответе очевидна и логика. Но мне логика непонятна. )

[Vadim Rublev]

> Но такая реализация не оптимальна, т.к. приводит к лишнему вызову функции indexPage, ...
Ясно.
У меня сейчас все работает, но маршруты сделаны (как уже говорил) "в куче" - вместе с обработчиками в main(). Я хочу обработчики вынести из main(), чтобы сделать "красиво как у всех", типа (функция Handle() или HandleFunc()):

И в каждом обработчике у меня есть http.FileServer(). Может быть это неоптимально...
Возможно, я вообще зря затеял вынос обработчиков из main()...

[Vadim Rublev]

> под "/" в mux.Handle("/", indexPage) попадут всё, что угодно, ...
Это ясно. Мне и надо, чтобы по всему что угодно (кроме другого, явно определенного) открывался index-файл. Кажется, это очевидно для любого сайта.. :) А другие ресурсы открываются по явно определенным маршрутам [которые приоритетнее "/"].

[Евгений Мамонов]

> И в каждом обработчике у меня есть http.FileServer(). Может быть это неоптимально...
так делать точно не стоит, т.к. это не правильно с точки зрения работы FileServer

> Возможно, я вообще зря затеял вынос обработчиков из main()...
точно не зря, потом, через время, вам или тому, кому ваш код достанется будет намного проще разобраться

Вот это тоже работать не будет, т.е. будет работать только первый, а до остальных очередь не дойдёт никогда

mux.HandleFunc("/", indexPage)
mux.HandleFunc("/", about)
mux.HandleFunc("/", contacts)
mux.HandleFunc("/", userPage)

Должно быть вот так

mux.HandleFunc("/about", about)
mux.HandleFunc("/contacts", contacts)
mux.HandleFunc("/user/{user_id}", userPage)
mux.HandleFunc("/", indexPage)

т.е. под "/" попадёт всё, что не попало выше.

Вижу, что у вас много вопросов и не складывается общая картина.
Думаю проще будет созвониться + могу посмотреть ваш код и рассказать как доработать чтобы получилось то, что вам нужно и почему именно так.

В общем если нужна помощь - напишите мне в телеграм Евгений Мамонов согласуем время.

[Vadim Rublev]

> Вот это тоже работать не будет, т.е. будет работать только первый, ...
Ну тут я ошибся, конечно. ))) Пример неправильно написал. Имел ввиду то, что вы поправили-написали.

> напишите мне в телеграм
Ок. Спасибо! :)

[Vadim Rublev]

...Написал вам в Телеге. Что-то тишина.. )

[Евгений Мамонов]

странно, телеграмм молчит
может не мне написали?
я сейчас свободен, можем сейчас созвониться
[phone] (Viber, WhatsApp)

[Евгений Мамонов]

EvgenyMamonov мой телеграм