RetroFed Forum
Your home away from modern life.
Bug Report: URI Sanitization
Post Reply
AuthorMessage
Pomal sh.itjust.works Pomal@sh.itjust.works
Joined: 2026-08-03
Quote

Morning Team,

Please see the thread found at this link, between myself and the OP:

https://sh.itjust.works/post/68082252

I’d like to request that the LinkResolver.ky file be updated so that URIs have their RFC-3986 well-defined Reserved Characters “percent-encoded” before being transferred to a buffer.

If you need an assist on dev for this, lmk :) I’m about since Gingerbread on developing for android, let alone kt but I have the spice😭

9   Original
Holla feddit.org waldfee@feddit.org
Joined: 2025-07-29
#2 · Quote Reply

Is this not an issue with the lemmy-ui? This issue describes a similar problem with links to posts from piefed users, which I made on lemmy iIrc, and the link in it still looks broken in the lemmy-ui.

Though I don't have the context of the file you pointed out; Github is down lol

2
Pomal sh.itjust.works Pomal@sh.itjust.works
Joined: 2026-08-03
#3 · (edited) Quote Reply

Is this not an issue with the lemmy-ui? This issue describes a similar problem with links to posts from piefed users, which I made on lemmy iIrc, and the link in it still looks broken in the lemmy-ui. Though I don't have the context of the file you pointed out; Github is down lol

From where I sit it appears to be the same issue, just upstream as well. Getting to speed on the internet is just gonna take time :)

Edit: same bug, same issue. The @ symbol means something to a url.

2
idunnololz Moderator idunnololz@lemmy.world
Joined: 2023-06-13
#4 · (edited) Quote Reply

I think this has been fixed in the latest release? Can you verify? Thanks.

Latest release should be v1.87.0.

1
Pomal sh.itjust.works Pomal@sh.itjust.works
Joined: 2026-08-03
#5 · (edited) Quote Reply

I think this has been fixed in the latest release? Can you verify? Thanks. Latest release should be v1.87.0.

@CorrectAlias@piefed.blahaj.zone mind giving it a whirl for testing?

Edit: I don’t use summit. I’m just an observer reporting.

2
Pomal sh.itjust.works Pomal@sh.itjust.works
Joined: 2026-08-03
#6 · Quote Reply

I think this has been fixed in the latest release? Can you verify? Thanks. Latest release should be v1.87.0.

Code block is unchanged, all I saw were some workflow changes and Open AI integrations being worked on.

1
idunnololz Moderator idunnololz@lemmy.world
Joined: 2023-06-13
#7 · (edited) Quote Reply

Code block is unchanged, all I saw were some workflow changes and Open AI integrations being worked on.

It was fixed at the start of August.

https://github.com/idunnololz/summit/commit/9fb33fdf7434621750d6a1b1a6b9f6ea778e412d

1
Pomal sh.itjust.works Pomal@sh.itjust.works
Joined: 2026-08-03
#8 · (edited) Quote Reply

It was fixed at the start of August. https://github.com/idunnololz/summit/commit/9fb33fdf7434621750d6a1b1a6b9f6ea778e412d

If you’re using your app, would you mind cross-linking a post from a separate instance?

1
idunnololz Moderator idunnololz@lemmy.world
Joined: 2023-06-13
#9 · (edited) Quote Reply

If you’re using your app, would you mind cross-linking a post from a separate instance?

Like as a test?

Here is a crosspost. Since the original bug is piefed related I made the crosspost from a piefed post:

https://lemmy.world/post/53309541

2
CorrectAlias piefed.blahaj.zone CorrectAlias@piefed.blahaj.zone
Joined: 2025-07-22
#10 · Quote Reply

@CorrectAlias@piefed.blahaj.zone mind giving it a whirl for testing? Edit: I don’t use summit. I’m just an observer reporting.

I'll check it out and report back, thanks!

2
CorrectAlias piefed.blahaj.zone CorrectAlias@piefed.blahaj.zone
Joined: 2025-07-22
#11 · Quote Reply

@CorrectAlias@piefed.blahaj.zone mind giving it a whirl for testing? Edit: I don’t use summit. I’m just an observer reporting.

On my end, it's the same for the crosspost links. I do think this is likely an upstream Piefed bug (that may have been slightly fixed), as when I'd copy the urls a week or so ago, it would also have the @ in the URL. Now though, this seems to not be the case.

2
Pomal sh.itjust.works Pomal@sh.itjust.works
Joined: 2026-08-03
#12 · Quote Reply

On my end, it's the same for the crosspost links. I do think this is likely an upstream Piefed bug (that may have been slightly fixed), as when I'd copy the urls a week or so ago, it would also have the @ in the URL. Now though, this seems to not be the case.

@idunnololz@lemmy.world

:)

1
idunnololz Moderator idunnololz@lemmy.world
Joined: 2023-06-13
#13 · (edited) Quote Reply

@idunnololz@lemmy.world :)

I think I misunderstood the bug. So the real bug is that the url shouldn't have an @? I tested in chrome/firefox and it works though so the url definitely seems to be valid.

I'd argue this is a bug on the other front ends for not supporting this url style.

4
Pomal sh.itjust.works Pomal@sh.itjust.works
Joined: 2026-08-03
#14 · Quote Reply

I think I misunderstood the bug. So the real bug is that the url shouldn't have an @? I tested in chrome/firefox and it works though so the url definitely seems to be valid. I'd argue this is a bug on the other front ends for not supporting this url style.

This is correct - the underlying linked url must be encoded. So the link that goes into the parenthesis’ ( ) .

Would you mind pointing me to an appropriate repo to delve?

1
Post Reply
Back to top