Page MenuHomeVyOS Platform
Create Task
Maniphest T1845

syslog host no longer accepts a port
Closed, ResolvedPublicBUG
Actions

Description

In crux/rolling, it's allowed to set a port for the syslog host. It appears that this isn't possible in Equulues

# set system syslog host 10.22.22.108:1514

  Invalid host FQDN or IP address
  Value validation failed
  Set failed

Details

Difficulty level
Normal (likely a few hours)
Version
1.3
Why the issue appeared?
Design mistake
Is it a breaking change?
Config syntax change (migratable)
Issue type
Bug (incorrect behavior)

Related Objects
Search...

Event Timeline

kroy created this task.Dec 4 2019, 5:06 AM
c-po added a subscriber: c-po.EditedDec 4 2019, 8:20 AM

Can this please be splittet in a host and a port node? Only WireGuard uses this notation all other services have a dedicated port node

hagbard claimed this task.Dec 4 2019, 4:09 PM
hagbard triaged this task as Normal priority.Dec 4 2019, 4:51 PM
hagbard changed Difficulty level from Unknown (require assessment) to Normal (likely a few hours).
hagbard changed Why the issue appeared? from Will be filled on close to Design mistake.
hagbard changed the task status from Open to In progress.Dec 4 2019, 4:58 PM
hagbard moved this task from Need Triage to In Progress on the VyOS 1.3 Equuleus board.
hagbard added a comment.Dec 4 2019, 5:52 PM

Actually I found out that the address:port wasn't implemented at all even if you were able to set it, it never was used within the config. I have that fixed now (not pushed yet). I also moved that part within the nodes, so it's going to be:

set system syslog global facility all level 'info'
set system syslog global facility protocols level 'debug'
set system syslog host 10.100.100.1 facility all level 'all'
set system syslog host 10.100.100.1 port '555'
set system syslog host 10.100.100.1 protocol 'tcp'
set system syslog host 10.100.100.2 facility kern level 'all'

Currently I check if facility and level is set, but I could make *.* as default target if it's not defined or so. Not sure if that would be necessary. I need to write a migration script only for the changed protocol node and if the syntax address:port exists, I gotta rewrite that then too.

hagbard added a project: VyOS 1.2 Crux (VyOS 1.2.5).Dec 4 2019, 5:54 PM
kroy added a comment.Dec 4 2019, 5:59 PM

It actually does work, if only by accident

host 10.22.22.108:1514 {
     facility all {
         level all
         protocol udp
     }
 }

Results in:

admin@edge:~$ cat /etc/rsyslog.d/vyos-rsyslog.conf 

## generated by syslog.py ##
## file based logging
$PreserveFQDN on
$outchannel global,/var/log/messages,262144,/usr/sbin/logrotate /etc/logrotate.d/vyos-rsyslog
*.info;local7.debug :omfile:$global
## remote logging
*.* @10.22.22.108:1514
hagbard added a comment.Dec 4 2019, 6:25 PM

Ah yes, it's taken entirely from the string, my fault I tested with the version you can only use an IP address.

c-po added a comment.Dec 4 2019, 6:27 PM

Thanks

hagbard changed the task status from In progress to Needs testing.Dec 4 2019, 11:40 PM
This comment was removed by hagbard.
hagbard mentioned this in T1828: Missing completion helper for "set system syslog host 192.0.2.1 facility all protocol".Dec 4 2019, 11:45 PM
hagbard changed the task status from Needs testing to In progress.Dec 5 2019, 4:07 PM
hagbard changed the task status from In progress to Needs testing.Dec 6 2019, 6:24 PM
This comment was removed by hagbard.
hagbard added a comment.Dec 6 2019, 7:45 PM

https://downloads.vyos.io/rolling/current/amd64/vyos-1.2-rolling-201912061907-amd64.iso and later include the fix

kroy added a comment.Dec 6 2019, 11:10 PM

Built a fresh rolling. It failed with:

[ system syslog host 10.22.22.108:1514 ]
Invalid host FQDN or IP address

[[system syslog]] failed
kroy added a comment.Dec 6 2019, 11:12 PM

Trying to apply the fix manually:

admin@edge# set system syslog host 10.22.22.108 port 1514
[edit]
admin@edge# commit
save
[ system syslog ]
Traceback (most recent call last):
  File "/usr/libexec/vyos/conf_mode/system-syslog.py", line 328, in <module>
    c = get_config()
  File "/usr/libexec/vyos/conf_mode/system-syslog.py", line 201, in get_config
    'proto': proto
UnboundLocalError: local variable 'proto' referenced before assignment

[[system syslog]] failed
Commit failed
[edit]
admin@edge#
hagbard added a comment.Dec 6 2019, 11:39 PM

fixed in http://dev.packages.vyos.net/repositories/current/vyos/pool/main/v/vyos-1x/vyos-1x_1.3.0-16_all.deb

pasik added a subscriber: pasik.Dec 8 2019, 7:00 PM
hagbard added a comment.Dec 10 2019, 6:00 PM

@kroy Please let me know if you still experience any issues (setting the port or migration).

kroy added a comment.Dec 10 2019, 7:09 PM

@hagbard Confirmed fix. Migration worked perfectly.

host 10.22.22.108:514 {
    facility all {
        level all
        protocol udp
    }
}

to

host 10.22.22.108 {
    facility all {
        level all
        protocol udp
    }
    port 514
}

and it looks like the rsyslogd.conf is correct:

*.* @10.22.22.108:514
hagbard closed this task as Resolved.Dec 10 2019, 7:10 PM
hagbard reopened this task as Backport candidate.
hagbard moved this task from In Progress to Finished on the VyOS 1.3 Equuleus board.
hagbard moved this task from Needs Triage to Backlog on the VyOS 1.2 Crux (VyOS 1.2.5) board.
hagbard added a project: Ready for Crux (1.2.x).Feb 8 2020, 4:13 PM
dmbaturin removed projects: Ready for Crux (1.2.x), VyOS 1.2 Crux (VyOS 1.2.5).Mar 9 2020, 12:38 PM
dmbaturin changed Is it a breaking change? from Unspecified (possibly destroys the router) to Config syntax change (migratable).
hagbard closed this task as Resolved.Mar 24 2020, 5:52 PM
erkin set Issue type to Bug (incorrect behavior).Aug 31 2021, 6:11 PM
syncer edited projects, added VyOS 1.3 Equuleus (1.3.0); removed VyOS 1.3 Equuleus.Aug 30 2022, 2:06 PM
syncer moved this task from Need Triage to Finished on the VyOS 1.3 Equuleus (1.3.0) board.
dmbaturin closed subtask T1850: syslog protocol can be set multiple times per facility for the same host as Resolved N/A.Jan 9 2024, 4:02 PM