Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sticker is not shown at matrix #38

Open
LuckyTurtleDev opened this issue Sep 6, 2023 · 8 comments
Open

sticker is not shown at matrix #38

LuckyTurtleDev opened this issue Sep 6, 2023 · 8 comments

Comments

@LuckyTurtleDev
Copy link
Owner

LuckyTurtleDev commented Sep 6, 2023

I tried it and compiled it but my stickers are not working (?) The media files appear to be there but nothing is showing, not even in Element. image

https://0x416c69.github.io/stickerpicker/web/#pack-tg_name_MyQuby (latest pack)

My synapse server is up to date. Is it required to enable dynamic_thumbnails in synapse for this to work? When trying to fetch those webp images, I'm getting this error 400:

{"errcode":"M_UNKNOWN","error":"Cannot find any thumbnails for the requested media ([b'ag5.ir', b'qAsAxJsWpWXAlGaJaeZrJwQC']). This might mean the media is not a supported_media_format=(image/jpeg, image/jpg, image/webp, image/gif, image/png) or that thumbnailing failed for some other reason. (Dynamic thumbnails are disabled on this server.)"}

Pack: https://t.me/addstickers/MyQuby


issue by @0x416c69 imported from maunium/stickerpicker#63 (comment)

@LuckyTurtleDev LuckyTurtleDev changed the title sticker is not showed at matrix sticker is not shown at matrix Sep 6, 2023
@LuckyTurtleDev
Copy link
Owner Author

@0x416c69 Which feature flags are enable and with mstickereditor version did you use?

@0x416c69
Copy link

0x416c69 commented Sep 6, 2023

@0x416c69 Which feature flags are enable and with mstickereditor version did you use?

I've downloaded the latest rust/cargo package with LLVM 16 (inside windows) and followed what was in the CI action job and compiled the master's HEAD (@ 76f1343)

I've also ran your unit tests, they've all succeeded in my machine (the machine that I've used to import those stickers)

There wasn't any errors or warnings when I executed the binary as well, it ran successfully

Edit: Do you have a working animated sticker so I can try it? Maybe the problem is with my animated sticker

@LuckyTurtleDev
Copy link
Owner Author

followed what was in the CI action job

So lottie feautre-flag should be disable. The mention pack is animated so import should not work. I think I have only disable the file conversion and do not create a warning/error if the lottie feature is disable (same issue for ffmpeg). So the original files (zstd compress lottie) was uploaded instead (which is not supported by element).

I should fix this behaviors.
They are three options:

  • uploading the (uncompressed) lottie/webm files
  • exiting with an error
  • create a warning and do not upload anything

I've also ran your unit tests, they've all succeeded in my machine (the machine that I've used to import those stickers)

Aslong you did not test ignored-test this do nothing. (cargo test --no-default-features --features ffmpeg,log -- --include-ignored but your telegram access token must be set as ENV for this)

@0x416c69
Copy link

0x416c69 commented Sep 6, 2023

(cargo test --no-default-features --features ffmpeg,log -- --include-ignored but your telegram access token must be set as ENV for this)

Should I run it and send you the output here?

@LuckyTurtleDev
Copy link
Owner Author

LuckyTurtleDev commented Sep 6, 2023

Should I run it and send you the output here?

This should not create useful data. Since lottie test are also disable if the lottie feautre-flag is not set.

But you can run it, to verify if video sticker and normal sticker works (keep in mind that this would also upload sticker to your matrix server). Using the binary to import sticker packs should also work.

@LuckyTurtleDev
Copy link
Owner Author

first patch is out b85d0d3. Animated sticker will be always unpack now. However since lottie and webm are not support by matrix client it would be a good idea to still create an warning/error.

I think the best way would be to replace the --noformat flag with --keep-webm and --keep-lottie. If the flag is not set a warning should be created and the sticker would be skipped.

@LuckyTurtleDev
Copy link
Owner Author

@0x416c69 can you retest the current main? Now an error is created if someone tries to import an animated/video sticker and the associated feature flag is disable.

@LuckyTurtleDev
Copy link
Owner Author

can this be closed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants