fix: webfinger
too many special cases just search username and domain there are indexes
This commit is contained in:
parent
f487ac06e9
commit
07d0d400d8
1 changed files with 29 additions and 48 deletions
|
@ -102,56 +102,37 @@ pub async fn webfinger(State(ctx): State<Context>, Query(query): Query<Webfinger
|
||||||
.replace("acct:", "")
|
.replace("acct:", "")
|
||||||
.split_once('@')
|
.split_once('@')
|
||||||
{
|
{
|
||||||
if domain == ctx.domain() {
|
let usr = model::actor::Entity::find()
|
||||||
// local user
|
.filter(model::actor::Column::PreferredUsername.eq(user))
|
||||||
let uid = ctx.uid(user);
|
.filter(model::actor::Column::Domain.eq(domain))
|
||||||
let usr = model::actor::Entity::find_by_ap_id(&uid)
|
.one(ctx.db())
|
||||||
.one(ctx.db())
|
.await?
|
||||||
.await?
|
.ok_or_else(UpubError::not_found)?;
|
||||||
.ok_or_else(UpubError::not_found)?;
|
|
||||||
|
|
||||||
Ok(JsonRD(JsonResourceDescriptor {
|
let expires = if domain == ctx.domain() {
|
||||||
subject: format!("acct:{user}@{domain}"),
|
// TODO configurable webfinger TTL, also 30 days may be too much???
|
||||||
aliases: vec![usr.id.clone()],
|
Some(chrono::Utc::now() + chrono::Duration::days(30))
|
||||||
links: vec![
|
|
||||||
JsonResourceDescriptorLink {
|
|
||||||
rel: "self".to_string(),
|
|
||||||
link_type: Some("application/ld+json".to_string()),
|
|
||||||
href: Some(usr.id),
|
|
||||||
properties: jrd::Map::default(),
|
|
||||||
titles: jrd::Map::default(),
|
|
||||||
},
|
|
||||||
],
|
|
||||||
expires: None,
|
|
||||||
properties: jrd::Map::default(),
|
|
||||||
}))
|
|
||||||
} else {
|
} else {
|
||||||
// remote user
|
// we are no authority on local users, this info should be considered already outdated,
|
||||||
let usr = model::actor::Entity::find()
|
// but can still be relevant, for example for our frontend
|
||||||
.filter(model::actor::Column::PreferredUsername.eq(user))
|
Some(chrono::Utc::now())
|
||||||
.filter(model::actor::Column::Domain.eq(domain))
|
};
|
||||||
.one(ctx.db())
|
|
||||||
.await?
|
|
||||||
.ok_or_else(UpubError::not_found)?;
|
|
||||||
|
|
||||||
Ok(JsonRD(JsonResourceDescriptor {
|
Ok(JsonRD(JsonResourceDescriptor {
|
||||||
subject: format!("acct:{user}@{domain}"),
|
subject: format!("acct:{user}@{domain}"),
|
||||||
aliases: vec![usr.id.clone()],
|
aliases: vec![usr.id.clone()],
|
||||||
links: vec![
|
links: vec![
|
||||||
JsonResourceDescriptorLink {
|
JsonResourceDescriptorLink {
|
||||||
rel: "self".to_string(),
|
rel: "self".to_string(),
|
||||||
link_type: Some("application/ld+json".to_string()),
|
link_type: Some("application/ld+json".to_string()),
|
||||||
href: Some(usr.id),
|
href: Some(usr.id),
|
||||||
properties: jrd::Map::default(),
|
properties: jrd::Map::default(),
|
||||||
titles: jrd::Map::default(),
|
titles: jrd::Map::default(),
|
||||||
},
|
},
|
||||||
],
|
],
|
||||||
properties: jrd::Map::default(),
|
properties: jrd::Map::default(),
|
||||||
// we are no authority on local users, this info should be considered already outdated,
|
expires,
|
||||||
// but can still be relevant, for example for our frontend
|
}))
|
||||||
expires: Some(chrono::Utc::now()),
|
|
||||||
}))
|
|
||||||
}
|
|
||||||
} else {
|
} else {
|
||||||
Err(StatusCode::UNPROCESSABLE_ENTITY.into())
|
Err(StatusCode::UNPROCESSABLE_ENTITY.into())
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue