Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Update to support 7.2 managed firewalls #301

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

antnsn
Copy link

@antnsn antnsn commented Oct 22, 2024

Workflow and Code Updates

  • Workflows:

    • node-version to 21 to ensure compatibility with the latest Node.js features and improvements.
    • the docker compose command to align with the updated syntax of the latest Compose version (docker compose build).
  • Changes:

    • /probe/managed_switch.go:
    • Updated the API call to fetch the managed switch status:
if err := c.Get("api/v2/monitor/switch-controller/managed-switch/status", "vdom=*&start=0&poe=true&port_stats=true&transceiver=true&count=1000", &response); err != nil {
    log.Printf("Error: %v", err)
}
  • Changes:
    • pkg/probe/managed_switch_test.go:
    • Adjusted the test to use the correct test data for validating the API response:
c.prepare("api/v2/monitor/switch-controller/managed-switch/status", "testdata/managed-switch.jsonnet")`

Dependency Updates

  • Addressed security vulnerabilities (CVEs) and ensured compatibility by updating the following dependencies:
  • Golang: Upgraded to 1.23, providing performance improvements and critical security fixes.
  • stdlib: Updated to 1.23 to maintain consistency with the latest Golang version and gain performance enhancements.
  • protobuf: Upgraded to 1.35.1 for bug fixes, security patches, and other enhancements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant