Schema $ids are declared under acs.org, a domain the project doesn't control #39
bharat-mukheja
started this conversation in
General
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
I'm a security engineer evaluating ACS from the enterprise-operator side — the case where the agent runtime belongs to a vendor and we're working out what we can actually enforce and log. While resolving the v0.1.0 schema package to validate some sample events, I hit something that looks unintentional.
specification/ACS/acs_schema.jsondeclares itself underagentcontrolstandard.org:But its
descriptionstates that each subschema "carries an$idunderhttps://acs.org/schema/v0.1.0/...", and that "validators that follow$idcan ignore this aggregator and load the modular files directly."acs.orgis the American Chemical Society. It's a different organisation that happens to share the acronym.Three consequences:
$id-following resolution can't work, by construction. The aggregator explicitly documents that path as supported — "validators that follow$idcan ignore this aggregator and load the modular files directly" — but nothing can be published atacs.org, so those identifiers will never resolve for anyone.$ref/$idsends traffic to a third party. Most validators don't fetch by default, so this isn't urgent, but the ones that do would hit an unrelated organisation's infrastructure, and the project has no control over what is served there now or later. Flagging in case you'd rather this went through private vulnerability reporting — I've assumed public is right since the fix is public and nothing here is exploitable.$refs have to be rewritten.Happy to open the PR. Three things I'd rather have from maintainers than assume:
agentcontrolstandard.orgthe long-term home, or is a dedicated schema domain planned? If the latter, better to move once than twice.$idacross the package is nominally breaking. In practice it breaks nothing, since nothing currently resolves — but whether this lands in v0.1.0 or bumps to v0.1.1 is a maintainer call, and I'd rather follow your convention than pick one.Agent-Control-Standard/ACSandGenAI-Security-Project/agent-control-standard. Separately,agentcontrolstandard.orglinks "Read the Specification" toaos.owasp.org, and its GitHub star badge countsOWASP/www-project-agent-observability-standard. If there's an intended canonical home, stating it in the README would save later contributors the same detour — and would tell me where to send the PR.All reactions