dandri/continuwuity
1
0
Fork 0
mirror of https://forgejo.ellis.link/continuwuation/continuwuity/ synced 2026-08-07 01:39:59 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

refactor: Remove redundant destination check in server auth logic

Browse Source 
Ruma already does this check for us
This commit is contained in:
Ginger
2026-07-26 20:20:11 -04:00
parent 61f6930b5d
commit 6115bb3101
1 changed files with 1 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/api/router/auth.rs
+1 -10
View File
@@ -129,15 +129,6 @@ async fn verify<R: IncomingRequest<Authentication = Self> + Any, B: AsRef<[u8]>
request: &hyper::Request<B>,
_query: AuthQueryParams,
) -> Result<Self::Identity> {
let destination = services.globals.server_name();
if output
.destination
.as_ref()
.is_some_and(|supplied_destination| supplied_destination != destination)
{
return Err!(Request(Unauthorized("Destination mismatch.")));
}
let key = services
.server_keys
.get_verify_key(&output.origin, &output.key)
@@ -149,7 +140,7 @@ async fn verify<R: IncomingRequest<Authentication = Self> + Any, B: AsRef<[u8]>
let keys: PubKeys = [(output.key.to_string(), key.key)].into();
let keys: PubKeyMap = [(output.origin.as_str().into(), keys)].into();
match output.verify_request(request, destination, &keys) {
match output.verify_request(request, services.globals.server_name(), &keys) {
| Ok(()) => {
if services
.moderation