Skip to content

🔐 v2.8.4 Improve API allowed instance keys settings. #202

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

Merged
merged 2 commits into from
Feb 3, 2025
Merged

Conversation

bmeares
Copy link
Owner

@bmeares bmeares commented Feb 3, 2025

v2.8.4

  • Allow for pattern matching in allowed_instance_keys.
    You may now generalize the instances exposed by the API by using Unix-style patterns in the list system:api:permissions:instances:allowed_instance_keys:

    {
      "api": {
        "permissions": {
          "instances": {
            "allowed_instance_keys": [
              "valkey:*",
              "*_dev"
            ]
          }
        }
      }
    }
  • Return pipe attributes for the route /pipes/{connector}/{metric}/{location}.
    The API routes /pipes/{connector}/{metric}/{location} and /pipes/{connector}/{metric}/{location}/attributes both return pipe attributes.

  • Check entire batches for verify rowcounts.
    The command verify rowcounts will now check batch boundaries before checking row-counts for individual chunks. This should moderately increase performance.

  • Kill orphaned child processes when the parent job is killed.
    Jobs created with pipeline arguments should now kill associated child processes.

  • Add --skip-hooks.
    The flag --skip-hooks prevents any sync hooks from firing when syncing pipes.

  • Fix allowed_instance_keys enforcement.

@bmeares bmeares self-assigned this Feb 3, 2025
@bmeares bmeares marked this pull request as ready for review February 3, 2025 23:50
@bmeares bmeares merged commit 229b9ab into main Feb 3, 2025
1 check passed
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