Merge pull request #3161 from systeembeheerder/systeembeheerder-patch…

…-fortios_acme Systeembeheerder patch fortios acme
ytti · May 15, 2024 · a14c06c · a14c06c
2 parents 9594ae1 + 05ed9f6
commit a14c06c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -13,6 +13,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 - xos: Hide radius and user secrets for XOS (@iriseden)
 - eos: Hide radius and snmp secrets for Arista EOS (@iriseden)
 - docker/podman: baseimage updated to phusion/baseimage:jammy-1.0.4
+- fortios: Hide date in acme certifcate comments (@systeembeheerder)
 
 ### Fixed
 - fixed prompt for vyos/vyatta to allow logins with non-priviliged accounts. Fixes #3111 (@h-lopez)

diff --git a/lib/oxidized/model/fortios.rb b/lib/oxidized/model/fortios.rb
@@ -77,6 +77,8 @@ class FortiOS < Oxidized::Model
       fullcfg = cmd(fullcmd)
       next if fullcfg.lines[1..3].join =~ /(Parsing error at|command parse error)/ # Don't show for unsupported devices (e.g. FortiAnalyzer, FortiManager, FortiMail)
 
+      fullcfg.gsub! /(set comments "Error \(No order found for account ID \d+\) on).*/, '\\1 <stripped>'
+
       cfg << fullcfg
       break
     end