Skip to content

[api] Wrong description on silo_user_list and silo_user_view #9359

@david-crespo

Description

@david-crespo

They say they're for built-in users, but it looks like they're for regular silo users users.

/// List built-in (system) users in silo
#[endpoint {
method = GET,
path = "/v1/system/users",
tags = ["system/silos"],
}]
async fn silo_user_list(
rqctx: RequestContext<Self::Context>,
query_params: Query<PaginatedById<params::SiloSelector>>,
) -> Result<HttpResponseOk<ResultsPage<views::User>>, HttpError>;
/// Fetch built-in (system) user
#[endpoint {
method = GET,
path = "/v1/system/users/{user_id}",
tags = ["system/silos"],
}]
async fn silo_user_view(
rqctx: RequestContext<Self::Context>,
path_params: Path<params::UserParam>,
query_params: Query<params::SiloSelector>,
) -> Result<HttpResponseOk<views::User>, HttpError>;

silo_user_list call chain

let users = nexus
.silo_list_users(&opctx, &silo_lookup, &pag_params)

paginated(silo_user, id, pagparams)
.filter(silo_id.eq(authz_silo_user_list.silo().id()))
.filter(user_provision_type.eq(db_silo.user_provision_type))
.filter(time_deleted.is_null())
.select(model::SiloUser::as_select())

silo_user_view call chain

let user = nexus
.silo_user_fetch(&opctx, &silo_lookup, path.user_id)

/// Fetch a user in a Silo
pub(crate) async fn silo_user_fetch(
&self,
opctx: &OpContext,
silo_lookup: &lookup::Silo<'_>,
silo_user_id: SiloUserUuid,
) -> LookupResult<SiloUser> {
let (authz_silo,) = silo_lookup.lookup_for(authz::Action::Read).await?;
let (_, silo_user) = self
.silo_user_lookup_by_id(
opctx,
&authz_silo,
silo_user_id,
authz::Action::Read,
)
.await?;
Ok(silo_user)
}

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions