Skip to content

Limit the information from the support script introduced in #1287 #1291

Closed

Description

@mattiaswal

Description

For security reasons limit the information given in show running, remove private keys, passwords and hashes.

For example ignore all nodes in the schema defined as default deny;

Additional Information

No response

General Information

Anyone can help out by sponsoring development of new features or contributing pull requests.
Please use this issue for discussions related to the feature.

Activity

  1. added
    triagePending investigation & classification (CCB)
    securityMedium or higher security impact
    on Dec 1, 2025
  2. troglobit commented on Dec 1, 2025

    @troglobit
    Contributor

    What would be great here is if we could have an operator level user

    Current state

    • Only one group: admin with full permit-all access
    • One specific deny rule for passwords (but admin can bypass it)
    • The nacm:default-deny-all extensions in YANG models would work, but only if there's a non-admin user

    What's needed

    1. operator group - Can read most config (sensitive data filtered out), can make configuration changes, but perhaps restricted from:
      - Reading private keys, passwords, shared secrets (respects nacm:default-deny-all)
      - Factory reset operations
      - User management
      - NACM rule changes
    2. guest (read-only/monitor) - Can only:
      - View operational state (interface status, routes, statistics)
      - Cannot read any configuration
      - Cannot make any changes

    This would make the support collect command much safer when run by an operator user - sensitive data would be automatically filtered by sysrepo's NACM implementation without needing any custom filtering code.

    The beauty of this approach is that it leverages the existing nacm:default-deny-all markings already present in the YANG models, making it maintainable and consistent with the NETCONF/YANG standards.

  3. added this to the Infix v26.02 milestone on Jan 26, 2026
  4. removed
    triagePending investigation & classification (CCB)
    on Jan 26, 2026
  5. modified the milestones: Infix v26.04, FUTURE on Apr 7, 2026
  6. modified the milestones: FUTURE, Infix v26.09 on Oct 5, 2026
  7. linked a pull request that will close this issueSupport collect #1632on Oct 5, 2026
  8. troglobit commented on Oct 5, 2026

    @troglobit
    Contributor

    Fixed on main @ 44a308a for v26.09

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    securityMedium or higher security impact

    Type

    No type

    Projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions