diff options
Diffstat (limited to 'lisp/erc/erc-networks.el')
-rw-r--r-- | lisp/erc/erc-networks.el | 57 |
1 files changed, 40 insertions, 17 deletions
diff --git a/lisp/erc/erc-networks.el b/lisp/erc/erc-networks.el index 19a7ab86437..fd8bed470ad 100644 --- a/lisp/erc/erc-networks.el +++ b/lisp/erc/erc-networks.el @@ -60,6 +60,7 @@ (declare-function erc-buffer-filter "erc" (predicate &optional proc)) (declare-function erc-current-nick "erc" nil) (declare-function erc-display-error-notice "erc" (parsed string)) +(declare-function erc-display-message "erc" (parsed type buffer msg &rest args)) (declare-function erc-error "erc" (&rest args)) (declare-function erc-get-buffer "erc" (target &optional proc)) (declare-function erc-server-buffer "erc" nil) @@ -1260,24 +1261,45 @@ given by the `RPL_ISUPPORT' NETWORK parameter." return name) (and-let* ((vanity (erc--get-isupport-entry 'NETWORK 'single)) ((intern vanity)))) + (erc-networks--id-given erc-networks--id) erc-networks--name-missing-sentinel)) -(defun erc-networks--set-name (_proc parsed) +(defvar erc-networks--allow-unknown-network nil + "Whether to ignore a failure in identifying the network. +If you need this as a user option, please say so via \\[erc-bug]. +Otherwise, expect it to vanish at any time.") ; Bug#59976 + +(defun erc-networks--set-name (proc parsed) "Set `erc-network' to the value returned by `erc-networks--determine'. -Signal an error when the network cannot be determined." +Print an error message when the network cannot be determined before +shutting down the connection." ;; Always update (possibly clobber) current value, if any. - (let ((name (erc-networks--determine))) - (when (eq name erc-networks--name-missing-sentinel) - ;; This can happen theoretically, e.g., if you're editing some - ;; settings interactively on a proxy service that impersonates IRC - ;; but aren't being proxied through to a real network. The + (pcase (setq erc-network (erc-networks--determine)) + ((and (pred (eq (erc-networks--id-given erc-networks--id))) + (let m (format "Couldn't determine network. Using given ID `%s'." + erc-network))) + (erc-display-message parsed 'notice nil m) + nil) + ((and + (guard (eq erc-network erc-networks--name-missing-sentinel)) + ;; This can happen theoretically, e.g., when adjusting settings + ;; on a proxy service that partially impersonates IRC but isn't + ;; currently conveying anything through to a real network. The ;; service may send a 422 but no NETWORK param (or *any* 005s). - (let ((m (concat "Failed to determine network. Please set entry for " - erc-server-announced-name " in `erc-networks-alist'."))) - (erc-display-error-notice parsed m) - (erc-error "Failed to determine network"))) ; beep - (setq erc-network name)) - nil) + (let m (concat "Failed to determine network. Please set entry for \"" + erc-server-announced-name "\" in `erc-networks-alist'" + " or consider calling `erc-tls' with the keyword `:id'." + " See Info:\"(erc) Network Identifier\" for more."))) + (require 'info) + (erc-display-error-notice parsed m) + (if erc-networks--allow-unknown-network + (progn + (erc-display-error-notice + parsed (format "Continuing anyway with network set to `%s'." + erc-network)) + nil) + (delete-process proc) + 'error)))) ;; This lives here in this file because all the other "on connect" ;; MOTD stuff ended up here (but perhaps that needs to change). @@ -1287,11 +1309,12 @@ Signal an error when the network cannot be determined." Copy source (prefix) from MOTD-ish message as a last resort." ;; The 004 handler never ran; see 2004-03-10 Diane Murray in change log (unless erc-server-announced-name - (erc-display-error-notice parsed "Failed to determine server name.") + (setq erc-server-announced-name (erc-response.sender parsed)) (erc-display-error-notice - parsed (concat "If this was unexpected, consider reporting it via " - (substitute-command-keys "\\[erc-bug]") ".")) - (setq erc-server-announced-name (erc-response.sender parsed))) + parsed (concat "Failed to determine server name. Using \"" + erc-server-announced-name "\" instead." + " If this was unexpected, consider reporting it via " + (substitute-command-keys "\\[erc-bug]") "."))) nil) (defun erc-unset-network-name (_nick _ip _reason) |