We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Registering a KAS to a platform currently requires a unique URI and a public_key that is either remote or cached:
URI
public_key
remote
cached
platform/service/policy/kasregistry/key_access_server_registry.proto
Line 22 in 0551247
Soon, it will also include (optionally) a common name for the KAS which is also unique: #153
However, the uri field in the proto above has no validation at present.
uri
./otdfctl policy kasr create -r https://remote.com/kas -u 'b' --json { "id": "87859c4d-b7f2-4366-8900-658b2a97de49", "uri": "b", // this should not be valid "public_key": { "PublicKey": { "Remote": "https://remote.com/kas" } }, "metadata": { "created_at": { "seconds": 1728938163, "nanos": 43877000 }, "updated_at": { "seconds": 1728938163, "nanos": 43877000 } } }
The text was updated successfully, but these errors were encountered:
fix(policy): enhance proto validation across policy requests (#1656)
df534c4
Resolves #1640 and a host of similar issues with validation throughout policy service protos
Successfully merging a pull request may close this issue.
Background
Registering a KAS to a platform currently requires a unique
URI
and apublic_key
that is eitherremote
orcached
:platform/service/policy/kasregistry/key_access_server_registry.proto
Line 22 in 0551247
Soon, it will also include (optionally) a common name for the KAS which is also unique: #153
However, the
uri
field in the proto above has no validation at present.Acceptance Criteria
The text was updated successfully, but these errors were encountered: