fix: prevent creating arp entries with empty mac
Some checks failed
continuous-integration/drone/push Build is failing
Some checks failed
continuous-integration/drone/push Build is failing
This commit is contained in:
@ -1,4 +1,4 @@
|
|||||||
use anyhow::Result;
|
use anyhow::{ensure, Result};
|
||||||
use lnvps_db::{Vm, VmIpAssignment};
|
use lnvps_db::{Vm, VmIpAssignment};
|
||||||
use rocket::async_trait;
|
use rocket::async_trait;
|
||||||
|
|
||||||
@ -30,6 +30,7 @@ pub struct ArpEntry {
|
|||||||
|
|
||||||
impl ArpEntry {
|
impl ArpEntry {
|
||||||
pub fn new(vm: &Vm, ip: &VmIpAssignment, interface: Option<String>) -> Result<Self> {
|
pub fn new(vm: &Vm, ip: &VmIpAssignment, interface: Option<String>) -> Result<Self> {
|
||||||
|
ensure!(vm.mac_address != "ff:ff:ff:ff:ff:ff", "MAC address is invalid because its blank");
|
||||||
Ok(Self {
|
Ok(Self {
|
||||||
id: ip.arp_ref.clone(),
|
id: ip.arp_ref.clone(),
|
||||||
address: ip.ip.clone(),
|
address: ip.ip.clone(),
|
||||||
@ -47,4 +48,3 @@ mod ovh;
|
|||||||
#[cfg(feature = "mikrotik")]
|
#[cfg(feature = "mikrotik")]
|
||||||
pub use mikrotik::MikrotikRouter;
|
pub use mikrotik::MikrotikRouter;
|
||||||
pub use ovh::OvhDedicatedServerVMacRouter;
|
pub use ovh::OvhDedicatedServerVMacRouter;
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user