diff options
| author | Jeff Carr <[email protected]> | 2025-09-07 20:34:56 -0500 |
|---|---|---|
| committer | Jeff Carr <[email protected]> | 2025-09-07 21:41:28 -0500 |
| commit | aef7b966ffc2ed4245713ec4d705eab22ecc9644 (patch) | |
| tree | 4bf1a9fabdb76f74c4fa6a264054f2f05ae9f810 | |
| parent | 4b78407d3f06b82e7a6f34911fa5b45b833cfeea (diff) | |
change the names from the http standardsv0.0.4
I'll probably regret this and if anyone ever sees this and
they actually use it, some people will probably complain. so: sorry
It's not like HTTP isn't really well documented these days :)
anyway, the *Addr fields are, I think, really really old
kinda dumb names that were made to be super unique and ackward
like RemoteAddr or whatever. Back then, having 'IP' would have
been a major problem and PITA and super confusing. So I think
everyone (I want to say 'we' but I don't know if I had anything
to do with RemoteAddr. I think I did add the Apache guys to
add SERVER_NAME support. Before that I don't think apache could
respond to mosaic as 2 different hostnames. My memory could be
incorrect.) named some of these things in ways that don't make
sense in a protobuf like this where it should simply be IP
| -rw-r--r-- | httpRequest.proto | 4 | ||||
| -rw-r--r-- | reqToPB.go | 16 |
2 files changed, 10 insertions, 10 deletions
diff --git a/httpRequest.proto b/httpRequest.proto index cc49b87..5b3d2c7 100644 --- a/httpRequest.proto +++ b/httpRequest.proto @@ -8,11 +8,11 @@ import "google/protobuf/timestamp.proto"; // Import the well-known type for Time message HttpRequest { // HttpRequest represents the essential fields of an incoming HTTP request. string method = 1; // The request method, e.g., "GET", "POST". - string url = 2; // The full URL of the request, including scheme, host, path, and query string. + string URL = 2; // The full URL of the request, including scheme, host, path, and query string. string route = 3; // just the route: "/add/" or "/find/" string proto = 4; // The protocol version, e.g., "HTTP/1.1", "HTTP/2.0". map<string, string> headers = 5; // The map of request headers. Header names are case-insensitive, - string remoteAddr = 6; // The remote IP address of the client, after resolving proxies. + string IP = 6; // The remote IP address of the client, after resolving proxies. string host = 7; // The host on which the URL is sought (www.wit.com) string hostname = 8; // The hostname of the client if passed from the client (mylaptop.fun.me) bytes body = 9; // The request body as raw bytes. @@ -58,14 +58,14 @@ func ReqToPB(r *http.Request) (*HttpRequest, error) { // log.Info("TRYING TO MARSHAL bytes:", len(msg), err) pb := &HttpRequest{ - Method: r.Method, - Url: r.URL.String(), - Proto: r.Proto, - Headers: headers, - RemoteAddr: getClientIP(r), - Host: r.Host, - Body: msg, - Hostname: r.Header.Get("hostname"), + Method: r.Method, + URL: r.URL.String(), + Proto: r.Proto, + Headers: headers, + IP: getClientIP(r), + Host: r.Host, + Body: msg, + Hostname: r.Header.Get("hostname"), } pb.Route = cleanURL(r.URL.Path) |
