30 May 2023 |
daedric | See those NaN ? | 16:08:32 |
daedric | The uploaded image info is similar to this: | 16:08:41 |
daedric | {
"type": "m.room.message",
"sender": "@dabot:aguiarvieira.pt",
"content": {
"body": "k6geqzxu013b1.png",
"msgtype": "m.image",
"filename": "k6geqzxu013b1.png",
"url": "mxc://aguiarvieira.pt/KZhnFisUFlnLUIcVMKeSHeGi"
},
"origin_server_ts": 1685461324850,
"unsigned": {
"age": 65
},
"event_id": "$Qbl9njTJFxGRs14168O4EFEtUeqpRIA0KJnkkKsxKws",
"room_id": "!VlADlaAKzYXdFhLQhZ:aguiarvieira.pt"
}
| 16:09:10 |
daedric | But, if i upload a image via Element, i get something like this: | 16:09:21 |
daedric | {
"type": "m.room.message",
"sender": "@daedric:aguiarvieira.pt",
"content": {
"body": "png-transparent-penguin-tux-design-logo-linux-penguin-animals-logo-vertebrate.png",
"info": {
"size": 18070,
"mimetype": "image/png",
"w": 920,
"h": 550,
"xyz.amorgan.blurhash": "LKRfd[ad.TtSS6bHs,jsyEkCMwad"
},
"msgtype": "m.image",
"url": "mxc://aguiarvieira.pt/gAasggirXwIdcCYjVyyUmsyp"
},
"origin_server_ts": 1685462095591,
"unsigned": {
"age": 102,
"transaction_id": "m1685462095526.38"
},
"event_id": "$QHBYeJYqXbwmJ0Q7y7anXP3IaaA0s9AJtGOTZgs3i8Y",
"room_id": "!BhYbDqbkGKjvDHvnTq:aguiarvieira.pt"
}
| 16:09:42 |
Midou | the "info" is missing from the json apparently | 16:10:17 |
tulir | info isn't required, but understandable bug in the archive | 16:11:07 |
TravisR | technically info is optional, so would be a regular bug in the archiver | 16:11:10 |
daedric | Result: | 16:13:07 |
daedric |  Download image.png | 16:13:40 |
daedric | But pressing on them, opens the full image. | 16:13:47 |
daedric | It's just the thumbnail generation that's buggy | 16:13:53 |
TravisR | yea, the archiver should switch to using /download or otherwise calculating its desired width/height | 16:14:31 |
TravisR | (it should be calculating that width/height already though... wonder why it uses info at all) | 16:14:47 |
daedric | Not complaining though. Quite amazing 🙂 | 16:16:12 |
daedric | Perhaps the info could be produced when uploading images via curl / api ? | 16:16:29 |
madlittlemods (Eric Eastwood) | In reply to @daedric:aguiarvieira.pt sent an image. Keep in mind that since the Matrix Public Archive uses Hydrogen under the hood, these are most likely just Hydrogen bugs and should be reported there | 16:45:42 |
daedric | Uhm... You're absolutly correct! | 16:50:30 |
daedric |  Download image.png | 16:50:56 |
31 May 2023 |
daedric | In reply to @travis:t2l.io yea, the archiver should switch to using /download or otherwise calculating its desired width/height I manage to fix it by calculating and including width (and hight and blurhash) on the script. Both the archiver and hydrogen no longer suffer from the "bug". | 07:45:33 |
| Andrew Morgan (anoa) changed their display name from Andrew Morgan (anoa) [away; back 31st] to Andrew Morgan (anoa). | 10:33:01 |
| Ștefan [back Tuesday] changed their display name from Ștefan to Ștefan [back Tuesday]. | 16:33:38 |
| @vagabondazulien:exp.farm left the room. | 21:32:57 |
MTRNord | I wish there was a project search room with reach other than this ^^ But did anyone by any chance write a Hubot adapter for matrix yet? https://hubot.github.com | 21:58:21 |
TravisR | https://github.com/davidar/hubot-matrix is the older one, but traditionally people just go with writing maubots for familiarity. | 22:04:58 |
TravisR | (found via #matrix-dev:matrix.org of all places) | 22:05:08 |
MTRNord | In reply to @travis:t2l.io https://github.com/davidar/hubot-matrix is the older one, but traditionally people just go with writing maubots for familiarity. thanks :) I wasnt able to find that one :D
Yeah I am wondering currently. I hit a case in my infra where basically all tools suck :) (Its always DNS! This time cert resolvers I use not working with multiple different authorative DNS servers at different providers for the same domain) So I might as well just write my own thing after all. But hubot seemed also interesting to try
| 22:06:35 |
MTRNord | but it only works with matrix integration :D | 22:06:45 |
MTRNord | * but it only works with matrix integration for me :D | 22:06:52 |
1 Jun 2023 |
| Mauro Romito (back on Monday) changed their display name from Mauro Romito to Mauro Romito (back on Monday). | 15:17:03 |