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

Fix escaping of special characters in tag names #3279

Open
roundhill opened this issue Dec 12, 2024 · 4 comments
Open

Fix escaping of special characters in tag names #3279

roundhill opened this issue Dec 12, 2024 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@roundhill
Copy link
Contributor

roundhill commented Dec 12, 2024

Please see Automattic/simplenote-android#1394 over at the Android repo. I think the web app is likely the cause, but it could be Android as well.

To Reproduce

  • Create a note with a tag on it with a dash, like test-ing.
  • Head over to the web and create another note, and use the test-ing tag. I did the tab autocomplete to add the tag.
  • You should now have two test-ing tags, but one of them is actually using a url escaped dash %2D.
@roundhill roundhill added the bug Something isn't working label Dec 12, 2024
@codebykat
Copy link
Member

@roundhill I have tried and tried but cannot seem to reproduce this with only the web app. I'd love to get to the bottom of it as it's been reported sporadically for years, but I'm not seeing it 😭

I have created tags with hyphens as well as a TON of other special characters. In your steps, did you create the initial tag on Android? Are we sure it isn't the Android app incorrectly escaping the tag name? Where are you seeing the screenshot of the incorrect tag name, was that the tag list on the web?

I inspected the tag in Simperium to verify it isn't getting encoded:

Image

As a side note, it's clear from my tags on my testing account that I've been trying to reproduce this bug for some years.

Image

@roundhill
Copy link
Contributor Author

roundhill commented Jan 15, 2025

Yeah, my bad there. The first step should be to create a note on another client. Here's what I just tested:

  1. Create a new note on iPhone with a new tag: jan-15.
  2. Switch to web app, observe note and tag are there.
  3. Create another note, add the tag by starting to type jan and then tab to autocomplete.
  4. You should now have two jan-15 tags in your tags list.

@roundhill
Copy link
Contributor Author

Maybe the bug is on iOS or Android, this is after creating a tag on iOS without messing with the web app 🤔

Image

@roundhill
Copy link
Contributor Author

Ok @jleandroperez informed me of this effort to escape the simperium key: Automattic/simplenote-ios#828

It looks like @sandymcfadden started on it here but it wasn't merged:
#3017

We need to ensure that the Simperium key gets percent-escaped in the same way that the other clients are doing it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants