fix asyncos prompt for hostnames containing "-"

Fixes #3327
ytti · Dec 6, 2024 · 37ae366 · 37ae366
1 parent 2955168
commit 37ae366
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -12,6 +12,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
 ### Fixed
 - tplink: send 'enable' before the enable password. Fixes #3271 (@robertcheramy)
+- asyncos: fix prompt for hostnames containing "-" . Fixes #3327 (@robertcheramy)
 
 
 ## [0.31.0 – 2024-11-29]

diff --git a/lib/oxidized/model/asyncos.rb b/lib/oxidized/model/asyncos.rb
@@ -2,7 +2,7 @@ class AsyncOS < Oxidized::Model
   using Refinements
 
   # ESA prompt "(mail.example.com)> " or "mail.example.com> "
-  prompt /^\r*([(]?[\w. ]+[)]?[#>]\s+)$/
+  prompt /^\r*\(?[\w.\- ]+\)?[#>]\s+$/
   comment '! '
 
   # Select passphrase display option

diff --git a/spec/model/asyncos.rb → spec/model/asyncos_spec.rb b/spec/model/asyncos.rb → spec/model/asyncos_spec.rb
@@ -11,7 +11,7 @@
   it 'matches different prompts' do
     _('(mail.example.com)> ').must_match AsyncOS.prompt
     _('mail.example.com> ').must_match AsyncOS.prompt
-    # Devices running in cluster mode (see issue #3327)
-    _('(Machine hostname) ').must_match AsyncOS.prompt
+    # Issue #3327
+    _('(Machine xx-xx-x1xx1-xxxx-xxxx.xxxx.xxxxx.example.xxx)> ').must_match AsyncOS.prompt
   end
 end