Почему выдает 500 ошибку go?

Question

[marselabdullin]

Переписал функции и выдает 500 ошибку теперь из-за нее, но причину не указывает:

func searchHandler(w http.ResponseWriter, r *http.Request) {
    u, err := url.Parse(r.URL.String())
    if err != nil {
        w.WriteHeader(http.StatusInternalServerError)
        w.Write([]byte("Internal server error"))
        return
    }

    params := u.Query()
    searchKey := params.Get("q")
    page := params.Get("page")
    if page == "" {
        page = "1"
    }


    search := &Search{}
    search.SearchKey = searchKey

    next, err := strconv.Atoi(page)
    if err != nil {
        http.Error(w, "Unexpected server error", http.StatusInternalServerError)
        return
    }

    search.NextPage = next
    pageSize := 20

    endpoint := fmt.Sprintf("https://newsapi.org/v2/everything?q=%s&pageSize=%d&page=%d&apiKey=%s&sortBy=publishedAt&language=en", url.QueryEscape(search.SearchKey), pageSize, search.NextPage, *apiKey)
    resp, err := http.Get(endpoint)
    if err != nil {
        w.WriteHeader(http.StatusInternalServerError)
        return
    }

    defer resp.Body.Close()

    if resp.StatusCode != 200 {
        w.WriteHeader(http.StatusInternalServerError)
        return
    }

    err = json.NewDecoder(resp.Body).Decode(&search.Results)
    if err != nil {
        w.WriteHeader(http.StatusInternalServerError)
        return
    }

    search.TotalPages = int(math.Ceil(float64(search.Results.TotalResults / pageSize)))
    err = tpl.Execute(w, search)
    if err != nil {
        w.WriteHeader(http.StatusInternalServerError)
    }

}

Comments

[d'Ivan]

http.Error(w, "Unexpected server error", http.StatusInternalServerError)

А почему не возвращать Bad request вместо 500?

Answer 1

[Алексей Маслов]

Вы везде проверяете на ошибку, но саму ошибку не выводите.
Можно, например, так:

http.Error(w, err.Error(), http.StatusInternalServerError)

Comments

[marselabdullin]

пишет 400 ошибка, я go просто решил посмотреть, а сам на python, поэтому плохо понимаю

[Алексей Маслов]

marselabdullin, на каком из условий кидает ошибку?
Посмотрите в дебагере что пишет.
Ошибки, конечно лучше логировать:
log.error, ну или воспользоваться сторонней библиотекой, например zap logger.

[marselabdullin]

Алексей Маслов, ну соответственно при проверке статуса вылетает приложения, то есть он некорректно осуществляет запрос, а как на go корректно я не знаю и делаю по туториалу, вроде должно работать

[Алексей Маслов]

marselabdullin, т.е. вылетает здесь?

if resp.StatusCode != 200 {
        w.WriteHeader(http.StatusInternalServerError)
        return
    }

Залогируйте и посмотрите какой статус код приходит. Или дебагером проверьте.
Да и вообще весь response можно проверить.
Мне кажется: что то не то с ApiKey.

[Алексей Маслов]

так де проверьте корректно ли формируется строка endpoint

Что кстати в apiKey содержится? Обычный string?

[marselabdullin]

Алексей Маслов, в apikey да стринг содержится, я не знаю как воспользоваться дебагером в данном случае

[Алексей Маслов]

marselabdullin, да просто выведете тогда в консоль или как вы там запускаете.
Например fmt.println(resp. StatusCode)
fmt.println(endpoint)

[marselabdullin]

ахха я протупил, да верно неправильно endpoint создался:

https://newsapi.org/v2/everything?q=&pageSize=20&p...

я делал запрос по адресу: 127.0.0.1:8000/search думал что так нужно делать запрос. Вывел endpoint как вы сказали и смотрю q - пустой, посмотрел откуда он берется и увидел, что из урла. Я изначально в туториале не понимал что это думал куки мб какие то. Потом до меня дошло, что это переменная в урле и запрос надо делать через поиск где она задается

[Алексей Маслов]

marselabdullin, отлично!