service identification by identifier

[raphael-goetz]

Resolves: #307

[github-actions]

GitLab Pipeline Action

General information

Link to pipeline: https://gitlab.com/code0-tech/development/aquila/-/pipelines/2493945972

Status: Passed
Duration: 2 minutes

Job summaries

docs:preview

Documentation preview available at https://code0-tech.gitlab.io/-/development/telescopium/-/jobs/14179496093/artifacts/out/index.html

[Copilot]

Pull request overview

This PR addresses issue #307 by changing service authorization from token-only checks to token+identifier checks, aiming to ensure a service is uniquely identified by both its configured identifier and its token.

Changes:

• Update server-side request handlers to validate requests using (token, identifier) rather than token alone.
• Extend ServiceConfiguration with identifier-aware has_service/has_runtime/has_action checks plus runtime identifier normalization logic.
• Add unit tests covering the new ServiceConfiguration matching rules.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
src/server/runtime_usage_service_server_impl.rs	Switches runtime usage authorization to has_service(token, ...) (currently with a hard-coded identifier).
src/server/runtime_status_service_server_impl.rs	Extracts runtime/action identifier from the status payload and authorizes via has_service(token, identifier).
src/server/module_service_server_impl.rs	Authorizes module updates using an identifier taken from the module list.
src/server/action_transfer_service_server_impl.rs	Tightens auth header validation and updates action authorization to use (token, identifier).
src/configuration/service.rs	Introduces identifier-aware authorization helpers, identifier normalization, and adds tests.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.