90
you are viewing a single comment's thread
view the rest of the comments
[-] nnrx@lemmy.world 2 points 6 months ago

Looks right here https://old.lemmy.world/u/e0qdk@reddthat.com

looks like the default sort for the user history api changed in whatever version of lemmy reddthat is running. Fix in the latest mlmym release 0.0.44 accounts for that now!

[-] e0qdk@reddthat.com 1 points 6 months ago

Thanks! I'll go ask Tiff about getting reddthat updated later.

BTW, is there a community for discussion of mlmym itself somewhere on lemmy? I can't participate on GitHub, but those aren't the only issues I've found. (e.g. there's also ?format=jpg&thumbnail=96 on non-pictrs links and a text handling issue with angle brackets...)

[-] nnrx@lemmy.world 2 points 6 months ago* (last edited 6 months ago)

no lemmy community for mlmym yet, that's good idea.

do you have an example of that thumbnail issue?

[-] e0qdk@reddthat.com 1 points 6 months ago* (last edited 6 months ago)

I'm having trouble finding an example of the thumbnail issue again right now but I was seeing the pictrs conversion parameters passed to URLs from catbox.moe, i.postimg.cc, and other sources in the CSS for the thumbnail when I reported the issue to Tiff ~3 weeks ago. It's possible that it got fixed/suppressed by another change since then though. (0.0.44 was deployed a few hours ago and I think there may have also been a beta patch bump for the lemmy backend at some point since I reported the issue originally in our local support community.)

I'll let you know if I see it pop up again.

For the text handling issue, I was seeing text like " by in " (i.e. "<thread title> by <username> in <community>" if it still happens) getting misinterpreted as raw HTML instead of being escaped. (i.e. <!-- raw HTML omitted --> was showing up in the HTML output for the page.)

You may recognize that text as the pattern for a recently fixed bug in the user profiles; I found the text handling issue while trying to explain the other issue to Tiff a few weeks ago.

Will edit this comment immediately after posting to let you know if I still see the text issue.

EDIT: I still see the text issue show up in this comment. https://old.reddthat.com/comment/10370610

[-] e0qdk@reddthat.com 1 points 5 months ago* (last edited 5 months ago)

I ran into an example of the thumbnail issue again today -- this time on a post from kbin: https://old.reddthat.com/post/19193476

The thumbnail looks like this in the HTML:

<div class="thumb">
  <a class="url"
     href="https://media.kbin.social/media/60/a4/60a45b8ff88b1b2e3a0f77b701feb323c5bbfb7ceeb75154ea7df5d6eea15ef8.jpg"
     >
    <div  style="background-image: url(https://media.kbin.social/media/60/a4/60a45b8ff88b1b2e3a0f77b701feb323c5bbfb7ceeb75154ea7df5d6eea15ef8.jpg?format=jpg&amp;thumbnail=96)"></div>
  </a>
</div>

Note that it's making a request to kbin.social with ?format=jpg&thumbnail=96 parameters in the CSS -- which results in the full image being loaded since kbin doesn't run pictrs.

The versions in use on reddthat (according to the settings page) are:

lemmy: 0.19.4-beta.7

mlmym: 0.0.44

[-] nnrx@lemmy.world 2 points 5 months ago

This kinda looks like an issue between lemmy and kbin. The issue is present on the lemmy ui as well. The ?format=jpg&thumbnail=96 parameter being there doesn't have an effect here. The issue is the thumbnail_url is reported wrong. You can see that here. The thumbnail_url should be: https://kbin.social/media/cache/resolve/entry_thumb/60/a4/60a45b8ff88b1b2e3a0f77b701feb323c5bbfb7ceeb75154ea7df5d6eea15ef8.jpg We could try and use some regex to lookout for kbin images and rewrite the thumbnail_url, but it should probably just be fixed upstream.

Found this github issue which might be related: https://github.com/LemmyNet/lemmy/issues/3775

this post was submitted on 09 May 2024
90 points (93.3% liked)

Technology

59144 readers
2645 users here now

This is a most excellent place for technology news and articles.


Our Rules


  1. Follow the lemmy.world rules.
  2. Only tech related content.
  3. Be excellent to each another!
  4. Mod approved content bots can post up to 10 articles per day.
  5. Threads asking for personal tech support may be deleted.
  6. Politics threads may be removed.
  7. No memes allowed as posts, OK to post as comments.
  8. Only approved bots from the list below, to ask if your bot can be added please contact us.
  9. Check for duplicates before posting, duplicates may be removed

Approved Bots


founded 1 year ago
MODERATORS