HoovesSocial/hippofish
2
0
Fork 0
Code Issues 3 Pull requests 1 Projects Releases Packages Wiki Activity Actions

fix(backend-rs): update padding logic to accomodate non-ASCII payload

Browse source
This commit is contained in:
Eana Hufwe 2024-06-05 20:30:41 +00:00
parent be894399a5
commit 7cf612f9f8
1 changed files with 7 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
packages/backend-rs/src/service/push_notification.rs
View file

@ -136,8 +136,9 @@ async fn encode_mastodon_payload(
serde_json::to_value(token.token)?,
);
// Some apps expect notification_id to be an integer, but never use it.
if ["IceCubesApp", "Mammoth", "feather", "MaserApp"].contains(&client.name.as_str()) {
// Some apps expect notification_id to be an integer,
// but doesn’t break when the ID doesn’t match the rest of API.
if ["IceCubesApp", "Mammoth", "feather", "MaserApp", "Metatext", "Feditext"].contains(&client.name.as_str()) {
let timestamp = object
.get("notification_id")
.and_then(|id| id.as_str())
@ -153,9 +154,11 @@ async fn encode_mastodon_payload(
// Adding space paddings to the end of JSON payload to prevent
// `esm` from adding null bytes payload which many Mastodon clients don’t support.
// https://firefish.dev/firefish/firefish/-/merge_requests/10905#note_6733
let padded_length = 126 + (res.len() + 1) / 128 * 128;
// not using {:padding_length$} directly on `res`` because we want the padding to be
// calculated based on the UTF-8 byte size of `res` instead of number of characters.
let padded_length = 126 + (res.len() + 1) / 128 * 128 - res.len();
Ok(format!("{:padded_length$}", res))
Ok(format!("{}{:padded_length$}", res, ""))
}
async fn handle_web_push_failure(