From: Siraaj Khandkar Date: Mon, 29 Nov 2021 14:54:32 +0000 (-0500) Subject: Organize User-Agent setting X-Git-Tag: 0.27.2 X-Git-Url: https://git.xandkar.net/?a=commitdiff_plain;h=7296ed949780b796bb0c529a857457ca710cf745;p=tt.git Organize User-Agent setting - type the components - handle missing peers in me file - handle multiple peers in me file --- diff --git a/info.rkt b/info.rkt index a31ac8e..49e0c96 100644 --- a/info.rkt +++ b/info.rkt @@ -6,7 +6,7 @@ (define pkg-desc "twtxt client") (define version - "0.27.1") + "0.27.2") (define pkg-authors '("Siraaj Khandkar ")) (define deps diff --git a/tt.rkt b/tt.rkt index 6ce0bd1..c29fabb 100644 --- a/tt.rkt +++ b/tt.rkt @@ -27,6 +27,18 @@ (∀ (α β) (U (cons 'ok α) (cons 'error β)))) +(struct User + ([uri : String] + [nick : (Option String)])) + +(struct User-Agent + ([user : User] + [prog : Prog])) + +(struct Prog + ([name : String] + [version : String])) + (struct Msg ([ts-epoch : Integer] [ts-orig : String] @@ -47,6 +59,72 @@ [body-input : Input-Port]) #:transparent) +(: prog Prog) +(define prog + (Prog "tt" (info:#%info-lookup 'version))) + +(: user-default User) +(define user-default + (User "https://github.com/xandkar/tt" #f)) + +(: user->str (-> User String)) +(define (user->str user) + (match-define (User u n) user) + (if n + (format "+~a; @~a" u n) + (format "+~a" u ))) + +(: user-agent->str (-> User-Agent String)) +(define (user-agent->str ua) + (match-define (User-Agent u p) ua) + (format "~a/~a (~a)" (Prog-name p) (Prog-version p) (user->str u))) + +(: user->user-agent User) +(define (user->user-agent user) + (User-Agent user prog)) + +(: user-agent-str String) +(define user-agent-str + (user-agent->str (user->user-agent user-default))) + +(: set-user-agent-str (-> Path-String Void)) +(define (set-user-agent-str filename) + (set! user-agent-str (user-agent->str (user->user-agent (file->user filename)))) + (log-info "User-Agent string is now set to: ~v" user-agent-str)) + +(: file->user (-> Path-String User)) +(define (file->user filename) + (if (file-exists? filename) + (match (set->list (file->peers filename)) + [(list p) + (log-info + "User-Agent. Found one peer in file: ~v. Using the found peer: ~a" + filename + (peer->str p)) + (peer->user p)] + [(list* p _) + (log-warning + "User-Agent. Multiple peers in file: ~v. Picking arbitrary: ~a" + filename + (peer->str p)) + (peer->user p)] + ['() + (log-warning + "User-Agent. No peers found in file: ~v. Using the default user: ~a" + filename + user-default) + user-default]) + (begin + (log-warning + "User-Agent. File doesn't exist: ~v. Using the default user: ~a" + filename + user-default) + user-default))) + +(define (peer->user p) + (match-define (Peer n _ u _) p) + (User u n)) + (: peers-equal? (-> Peer Peer Boolean)) (define (peers-equal? p1 p2) (equal? (Peer-uri-str p1) @@ -492,21 +570,6 @@ [_ #f])) -(: user-agent String) -(define user-agent - (let* - ([prog-name "tt"] - [prog-version (info:#%info-lookup 'version)] - [prog-uri "https://github.com/xandkar/tt"] - [user-peer-file (build-path tt-home-dir "me")] - [user - (if (file-exists? user-peer-file) - (match (set-first (file->peers user-peer-file)) - [(Peer #f _ u _) (format "+~a" u )] - [(Peer n _ u _) (format "+~a; @~a" u n)]) - (format "+~a" prog-uri))]) - (format "~a/~a (~a)" prog-name prog-version user))) - (: header-get (-> (Listof Bytes) Bytes (Option Bytes))) (define (header-get headers name) (match (filter-map (curry extract-field name) headers) @@ -599,7 +662,7 @@ (define-values (status-line headers body-input) (http-sendrecv/url u - #:headers (list (format "User-Agent: ~a" user-agent)))) + #:headers (list (format "User-Agent: ~a" user-agent-str)))) `(ok . ,(Resp status-line headers body-input)))) (channel-put result-chan result)))) (define result @@ -915,6 +978,7 @@ #:args (command . args) (define log-writer (log-writer-start log-level)) (current-command-line-arguments (list->vector args)) + (set-user-agent-str (build-path tt-home-dir "me")) ; TODO dispatch should return status with which we should exit after cleanups (dispatch command) (log-writer-stop log-writer))))