Skip to content
This repository has been archived by the owner on Dec 1, 2023. It is now read-only.

Sending none instead of empty string for interfaces #132

Closed
ma9icmarker opened this issue May 22, 2023 · 1 comment
Closed

Sending none instead of empty string for interfaces #132

ma9icmarker opened this issue May 22, 2023 · 1 comment
Labels
status: accepted This issue has been accepted by the maintainers team for implementation

Comments

@ma9icmarker
Copy link

Interface objects populated from IPFabric seems to hit an error with device imports. When there is no description on the device, the plug-in should send an empty string for the interface, as per the second example below.

interface_obj.description = fields.get ("description")

interface_obj.description = fields.get ("description", “”)

@ghost
Copy link

ghost commented May 23, 2023

Should this be changed to also?
description=iface.get("dscr", ""),

description=interface_record.description if interface_record.description else None,

description=interface_record.description if interface_record.description else "",

@pke11y pke11y added the status: accepted This issue has been accepted by the maintainers team for implementation label May 25, 2023
@alhogan alhogan closed this as completed Sep 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: accepted This issue has been accepted by the maintainers team for implementation
Projects
None yet
Development

No branches or pull requests

3 participants