-
Notifications
You must be signed in to change notification settings - Fork 565
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
Comments
@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: 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. |
Yeah, my bad there. The first step should be to create a note on another client. Here's what I just tested:
|
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: We need to ensure that the Simperium key gets percent-escaped in the same way that the other clients are doing it. |
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
test-ing
.test-ing
tag. I did the tab autocomplete to add the tag.test-ing
tags, but one of them is actually using a url escaped dash%2D
.The text was updated successfully, but these errors were encountered: