Login seems to work

2025-03-05 09:21:37 -05:00 · 2020-12-01 03:24:03 -06:00 · 2020-12-01 03:24:03 -06:00 · 9b49bd82a2
commit 9b49bd82a2
parent e6f2f4076e
4 changed files with 56 additions and 38 deletions
--- a/ement-api.el
+++ b/ement-api.el
@ -39,6 +39,7 @@
 (require 'plz)

 (require 'ement-macros)
+(require 'ement-structs)

 ;;;; Variables

@ -51,15 +52,17 @@

 ;;;; Functions

-(cl-defun ement-api (hostname port token endpoint then
+(cl-defun ement-api (server token endpoint then
                            &key _timeout data params
                            (content-type "application/json")
                            (else #'ement-api-error) (method 'get)
                            (json-read-fn #'json-read))
+  "FIXME: Docstring."
  ;; FIXME: Use transaction-id or add it in calling functions.
  ;; FIXME: Use timeout.
  (declare (indent defun))
-  (pcase-let* ((path (concat "/_matrix/client/r0/" endpoint))
+  (pcase-let* (((cl-struct ement-server hostname port) server)
+               (path (concat "/_matrix/client/r0/" endpoint))
 	       (query (url-build-query-string params))
 	       (filename (concat path "?" query))
               (url (url-recreate-url
@ -70,9 +73,7 @@
    ;; function on the session object, which may be very large, it
    ;; will take a very long time to print into the warnings buffer.
    ;;  (debug-warn (current-time) method url headers)
-    (pcase-exhaustive method
-      ('get (plz-get url :headers headers :as json-read-fn :then then :else else))
-      ('put (plz-put url data :headers headers :as json-read-fn :then then :else else)))))
+    (plz method url :headers headers :body data :as json-read-fn :then then :else else)))

 (defun ement-api-error (&rest args)
  "Signal an error about ARGS."
--- a/ement-room.el
+++ b/ement-room.el
@ -47,6 +47,7 @@
 (defvar-local ement-session nil
  "Ement session for current buffer.")

+(declare-function ement-view-room "ement.el")
 (defvar ement-room-mode-map
  (let ((map (make-sparse-keymap)))
    (define-key map (kbd "g") #'ement-room-sync)
@ -141,16 +142,15 @@ See Info node `(elisp)Other Display Specs'."
 			 (read-number "Number of messages: ")
 		       10)))
  (pcase-let* (((cl-struct ement-session server token) session)
-	       ((cl-struct ement-server hostname port) server)
 	       ((cl-struct ement-room id prev-batch) room)
 	       (endpoint (format "rooms/%s/messages" (url-hexify-string id))))
-    (ement-api hostname port token endpoint
+    (ement-api server token endpoint
      (apply-partially #'ement-room-retro-callback room)
      :params (list (list "from" prev-batch)
 		    (list "dir" "b")
 		    (list   "limit" (number-to-string number))))))

-
+(declare-function ement--make-event "ement.el")
 (defun ement-room-retro-callback (room data)
  "Push new DATA to ROOM on SESSION and add events to room buffer."
  (pcase-let* (((cl-struct ement-room) room)
@ -203,13 +203,12 @@ See Info node `(elisp)Other Display Specs'."
  (let ((body (read-string "Send message: ")))
    (unless (string-empty-p body)
      (pcase-let* (((cl-struct ement-session server token) ement-session)
-                   ((cl-struct ement-server hostname port) server)
                   ((cl-struct ement-room id) ement-room)
                   (endpoint (format "rooms/%s/send/%s/%s" (url-hexify-string id)
 				     "m.room.message" (cl-incf (ement-session-transaction-id ement-session))))
 		   (json-string (json-encode (ement-alist "msgtype" "m.text"
 							  "body" body))))
-        (ement-api hostname port token endpoint
+        (ement-api server token endpoint
          (lambda (&rest args)
            (message "SEND MESSAGE CALLBACK: %S" args))
 	  :data json-string
--- a/ement-structs.el
+++ b/ement-structs.el
@ -41,7 +41,7 @@
  hostname port)

 (cl-defstruct ement-session
-  user server token transaction-id rooms next-batch)
+  user server token transaction-id rooms next-batch device-id initial-device-display-name)

 (cl-defstruct ement-room
  id display-name prev-batch
--- a/ement.el
+++ b/ement.el
@ -72,6 +72,10 @@
 (defvar ement-progress-value nil
  "Used to report progress while processing sync events.")

+(defvar ement-login-hook '(ement--sync)
+  "Hook run after successful login.
+Run with one argument, the session logged into.")
+
 (defvar ement-sync-callback-hook '(ement--update-room-buffers ement--auto-sync)
  "Hook run after `ement--sync-callback'.
 Hooks are called with one argument, the session that was
@ -103,33 +107,48 @@ synced.")

 ;;;; Commands

-(defun ement-connect (user-id _password hostname token &optional transaction-id)
-  ;; FIXME: Use password if given.
-  "Connect to Matrix and sync once."
-  (interactive (pcase-let* (((map username server token ('txn-id transaction-id))
-                             (ement--load-session)))
-                 (list username nil server token transaction-id))
-               ;; (list (read-string "User ID: " (or (when (car ement-sessions)
-               ;;                                      (ement-session-user (car ement-sessions)))
-               ;;                                    ""))
-               ;;       (read-passwd "Password: ")
-               ;;       (read-string "Hostname (default: from user ID): ")
-               ;;       (alist-get 'token (ement--load-session)))
-               )
-  ;; FIXME: Overwrites any current session.
-  (pcase-let* ((hostname (if (not (string-empty-p hostname))
-                             hostname
-                           (if (string-match (rx ":" (group (1+ anything))) user-id)
+(defun ement-connect (user-id password &optional token transaction-id)
+  "Connect to Matrix.
+Interactively, with prefix, ignore a saved session and log in again."
+  (interactive (pcase-let* (((map username password server token ('txn-id transaction-id))
+                             (or (unless current-prefix-arg
+				   (ement--load-session))
+                                 (ement-alist 'username (read-string "User ID: ")
+                                              'password (read-passwd "Password: ")))))
+                 (list username password token transaction-id)))
+  (pcase-let* ((hostname (if (string-match (rx ":" (group (1+ anything))) user-id)
                             (match-string 1 user-id)
-                             "matrix.org")))
+                           "matrix.org"))
               ;; FIXME: Lookup hostname from user ID with DNS.
               ;; FIXME: Dynamic port.
               (server (make-ement-server :hostname hostname :port 443))
               (user (make-ement-user :id user-id))
               (transaction-id (or transaction-id (random 100000)))
               (session (make-ement-session :user user :server server :token token :transaction-id transaction-id)))
-    (setf ement-sessions (list session)))
+    (if token
+        (progn
+          ;; FIXME: Overwrites any current session.
+          (setf ement-sessions (list session))
          (ement--sync (car ement-sessions)))
+      ;; Log in.
+      (pcase-let* (((cl-struct ement-session user server token device-id initial-device-display-name) session)
+                   ((cl-struct ement-user id) user)
+                   (data (ement-alist "type" "m.login.password"
+                                      "user" id
+                                      "password" password
+                                      "device_id" device-id
+                                      "initial_device_display_name" initial-device-display-name)))
+        ;; TODO: Clear password in callback.
+        (ement-api server token "login" (apply-partially #'ement--login-callback session)
+          :data (json-encode data) :method 'post)))))
+
+(defun ement--login-callback (session data)
+  "Finish logging in to SESSION and sync."
+  (pcase-let* (((map ('access_token token) ('device_id device-id)) data))
+    (setf ement-sessions (list session)
+	  (ement-session-token session) token
+          (ement-session-device-id session) device-id)
+    (run-hook-with-args 'ement-login-hook session)))

 (defun ement-view-room (session room)
  "Switch to a buffer showing ROOM on SESSION."
@ -162,7 +181,6 @@ If SESSION has a `next-batch' token, it's used."
  ;; TODO: Filtering: <https://matrix.org/docs/spec/client_server/r0.6.1#filtering>.
  (cl-assert (not (map-elt ement-syncs session)))
  (pcase-let* (((cl-struct ement-session server token next-batch) session)
-               ((cl-struct ement-server hostname port) server)
               (params (remove nil (list (list "full_state" (if next-batch "false" "true"))
 					 (when next-batch
 					   (list "since" next-batch))
@ -170,7 +188,7 @@ If SESSION has a `next-batch' token, it's used."
 					   (list "timeout" "30000")))))
               (sync-start-time (time-to-seconds))
               ;; FIXME: Auto-sync again in error handler.
-               (process (ement-api hostname port token "sync" (apply-partially #'ement--sync-callback session)
+               (process (ement-api server token "sync" (apply-partially #'ement--sync-callback session)
                          :params params
                          :else (lambda (plz-error)
                                  (setf (map-elt ement-syncs session) nil)