Skip to content

Narrow down switch API. - #193

Draft
Gerrit91 wants to merge 1 commit into
mainfrom
switch-api
Draft

Gerrit91 wants to merge 1 commit into
mainfrom
switch-api

Conversation

@Gerrit91

Copy link
Copy Markdown
Contributor

Description

To me it feels like the switch API is a bit too broad and there are no descriptions anywhere what fields are used in which circumstances, potentially even leading to possibilities to misuse the API.

Here is an idea to narrow down the API slightly:

  1. Disallow admins to send nics and os, these come from register (also remove duplicate updated_at timestamp?)
  2. Create a register request that really only contains the fields that the metal-core is expected to set

Used AI-Tools ✨

  • None used for generation

@Gerrit91

Gerrit91 commented Sep 21, 2026

Copy link
Copy Markdown
Contributor Author

This results in quite huge server changes (I tried already), so I am unsure if we really want this. But somehow I am not 100% convinced by how the API was defined either. :s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant