`http.Client.Do()`メソッドを使う際、既存のリクエストを再利用する方法を教えてください。

###前提・実現したいこと
Go言語のGorillaを使ってセッションを管理しています。
クライアント側からサーバ側に新しいリクエストを生成して送信した場合、前回保存したセッション情報を読み込むことができませんでした。
この場合、セッションの維持する方法を教えてください。

###発生している問題・エラーメッセージ

エラーメッセージは特にありません。

###該当のソースコード

go
1// Client側
2package main
3
4import (
5	"io/ioutil"
6	"log"
7	"net/http"
8
9	"github.com/gorilla/mux"
10	"github.com/gorilla/securecookie"
11	"github.com/gorilla/sessions"
12)
13
14var (
15	store       *sessions.CookieStore = sessions.NewCookieStore(securecookie.GenerateRandomKey(64))
16	session     *sessions.Session
17	SessionName = "client"
18)
19
20func main() {
21	store.Options = &sessions.Options{
22		Path:     "/",
23		MaxAge:   60 * 15,
24		Secure:   false,
25		HttpOnly: true,
26	}
27	router := mux.NewRouter()
28	router.HandleFunc("/", cIndex)
29	router.HandleFunc("/test", cTest)
30	http.ListenAndServe(":7000", router)
31}
32
33func cIndex(w http.ResponseWriter, r *http.Request) {
34	session, err := store.Get(r, SessionName)
35	if err != nil {
36		log.Println("Client Error 1 : ", err.Error())
37	}
38	session.Values["foo"] = "bar"
39	session.Save(r, w)
40	w.Header().Set("Location", "http://localhost:8080?foo=bar")
41	w.Header().Set("Content-Type", "application/x-www-form-urlencoded")
42	w.WriteHeader(http.StatusFound)
43
44}
45
46func cTest(w http.ResponseWriter, r *http.Request) {
47	req, err := http.NewRequest("GET", "http://localhost:8080/test", nil)
48	if err != nil {
49		log.Println("Client Error 2 : ", err.Error())
50	}
51	req.SetBasicAuth("sample_id", "sample_secret")
52	// req.Header.Set("Location", provider.TokenEP+"?"+t.encode())
53	req.Header.Set("Content-Type", "application/x-www-form-urlencoded")
54	c := &http.Client{
55		Transport: &http.Transport{},
56	}
57	resp, err := c.Do(req)
58	if err != nil {
59		log.Println("Client Error 3 : ", err.Error())
60	}
61	defer resp.Body.Close()
62	body, err := ioutil.ReadAll(resp.Body)
63	if err != nil {
64		log.Println("Client Error 4 : ", err.Error())
65	}
66	log.Println(string(body))
67}

// サーバ側
package main

import (
	"encoding/json"
	"log"
	"net/http"

	"github.com/gorilla/mux"
	"github.com/gorilla/securecookie"
	"github.com/gorilla/sessions"
)

var (
	store       *sessions.CookieStore = sessions.NewCookieStore(securecookie.GenerateRandomKey(64))
	session     *sessions.Session
	SessionName = "server"
)

func main() {
	store.Options = &sessions.Options{
		Path:     "/",
		MaxAge:   60 * 15,
		Secure:   false,
		HttpOnly: true,
	}
	router := mux.NewRouter()
	router.HandleFunc("/", sIndex)
	router.HandleFunc("/test", sTest)
	http.ListenAndServe(":8080", router)
}

func sIndex(w http.ResponseWriter, r *http.Request) {
	r.ParseForm()
	v := r.FormValue("foo")
	session, err := store.Get(r, SessionName)
	if err != nil {
		log.Println("Server Error 1 : ", err.Error())
	}
	session.Values["foo"] = v
	session.Save(r, w)
	w.Header().Set("Location", "http://localhost:7000/test")
	w.Header().Set("Content-Type", "application/x-www-form-urlencoded")
	w.WriteHeader(http.StatusFound)
}

func sTest(w http.ResponseWriter, r *http.Request) {
	session, err := store.Get(r, SessionName)
	if err != nil {
		log.Println("Server Error 2 : ", err.Error())
	}
	v, ok := session.Values["foo"].(string)
	if !ok {
		log.Printf("foo = %v\n", v)
	}
	data := struct {
		Foo string
	}{
		v, // I want v is bar
	}

	bytes, err := json.Marshal(data)
	if err != nil {
		log.Println("Server Error 3 : ", err.Error())
	}
	w.Header().Set("Content-Type", "application/json")
	w.WriteHeader(http.StatusOK)
	w.Write(bytes)
}

###試したこと
① クライアント側でhttp.Clientを生成する時、http.Transportも生成してリクエストを送信する。(c := &http.Client{Transport:&http.Transport{}})
② リクエスト生成後、コンテキストを使用してリクエスト送信する。(req = request.WithContext(context.BackgroundContext()))

###補足情報(言語/FW/ツール等のバージョンなど)
20170704 追記
タイトル選択にミスがあり、修正しました。
私が聞きたかった質問はリクエストを再利用する方法でした。
試したコードで動作するのは確認しましたが、これよりいい方法があれば教えてください。

go
1// Client
2func cTest(w http.ResponseWrite, r *http.Request) {
3    serverURL := "http://localhost:8080/test"
4    r.Method = http.MethodGet
5    r.Host = serverURL  // server
6    v, err := url.Parse(serverURL)
7    if err != nil {
8        log.Println("url parsing error occurred : ", err.Error())
9        return
10    }
11    c := &http.Client{Transport: &http.Transport{}}
12    resp, err := c.Transport.RoundTrip(r)
13    if err != nil {
14        log.Println("client roundtrip error occurred : ", err.Error())
15        return
16    }
17    defer resp.Body.Close()
18    body, err := ioutil.ReadAll(resp.Body)
19    if err != nil {
20        log.Println("response body reading error occurred : ", err.Error())
21        return
22    }
23    w.Write(body)
24}

行動規範の内容に同意します

回答2件

ベストアンサー

下記の方法を使えば、サーバ側で以前設定したセッションを使うことが出来ました。

// この以前にClient・サーバの間データをやり取りして、サーバのセッションにデータを保存しておきました。
// Client
func cTest(w http.ResponseWriter, r *http.Request) {
    serverURL := "https://localhost:8080/test"
    r.Method = http.MethodGet
    r.Host = serverURL  // server
    v, err := url.Parse(serverURL)
    if err != nil {
        log.Println("url parsing error occurred : ", err.Error())
        return
    }
    r.URL = v
    c := &http.Client{ 
        // 開発環境では自己証明した証明書を使いますので、証明書を無視するコードが書いてあります。
        Transport: &http.Transport{
            TLSClientConfig: &tls.Config{InsecureSkipVerify: true},
        },
    }
    resp, err := c.Transport.RoundTrip(r)
    if err != nil {
        log.Println("client roundtrip error occurred : ", err.Error())
        return
    }
    defer resp.Body.Close()
    body, err := ioutil.ReadAll(resp.Body)
    if err != nil {
        log.Println("response body reading error occurred : ", err.Error())
        return
    }
    w.Write(body)
}

しかし、mattnさんの説明を読んで、クライアント側でhttp.Clientを使いまわせば、より良い方法で同じ結果を得ることが出来るかと思います。

テストして追記します。

投稿2017/07/12 13:55

編集2017/07/13 01:13

退会済みユーザー

総合スコア0

mattn

2017/07/12 14:52 編集

Transport の差し替えはなぜ必要なのですか？もう少しいうとリクエストを再利用する理由が分かりません。

退会済みユーザー

2017/07/12 15:04 編集

コードが詳しく書いてなかったですね。すみません。私が書いている実際のコードではTLSを使っています。 Transportに下の TLSClientConfig: &tls.Config{InsecureSkipVerify: true} コードを追加しないとエラーが発生するので、Transportを差し替えました。また、クライアント側からサーバにcURLのようにリクエストを送信したいと思っていて、その間セッションも維持したいと思ってリクエストを再利用しようと思いました。

mattn

2017/07/12 15:23 編集

http.Request はステートを持ちません。なのでもともと再利用できるように設計されています。セッションを保持しているのはクッキー(サーバがクッキーセッションを使っておられる所から分かります)であって、「セッションを保持する」に対してやるべきは http.Client.Jar を使う事の様に思います。現状、質問に載せて頂いているコードではセッションは保持しない様に思えます。追記: あと、クッキーが保存されるのは http.Client であって Transport ではないので、http.Client を使いまわさない限り前回のセッションを引き継ぐ事は出来ません。さらに追記: しつこい様ですみません。何故ここまで言うのかというと Google 検索で「リクエストを再利用をする」等をキーワードにこの回答を見つけてきた人が「下の方法を参考にすればと思います」と書かれてしまっているのを信じてしまうのが困るからです。

退会済みユーザー

2017/07/13 01:01

mattnさん、詳細な説明ありがとうございます！最初からhttp.Clientを使いまわすように設計しないといけないのですね。クライアント側のコードを修正してみます。私は今までクライアント側もサーバのように作成しました。解決方法のところは修正します。

行動規範の内容に同意します

サーバ側のコードでセッションクッキーを使っておられるので、当然ですがクライアントがクッキー対応している必要があります。以下の様に cookiejar を付けて http.Client を作成するとクッキー対応できます。
この client を使いまわせばセッションが有効になるかと思います。

golang
1package main
2
3import (
4	"log"
5	"net/http"
6	"net/http/cookiejar"
7)
8
9func main() {
10	jar, err := cookiejar.New(nil)
11	if err != nil {
12		log.Fatal(err)
13	}
14
15	client := &http.Client{
16		Jar: jar,
17	}
18
19	if _, err = client.Get("http://example.com/"); err != nil {
20		log.Fatal(err)
21	}
22	// この後も client を使う
23}