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

Special character is synced wrong #222

Closed
linuxrecon opened this issue Jan 15, 2019 · 4 comments
Closed

Special character is synced wrong #222

linuxrecon opened this issue Jan 15, 2019 · 4 comments
Labels
Projects

Comments

@linuxrecon
Copy link

Software versions

  • Browser: Firefox 64
  • Nextcloud: 15.0.0
  • Nextcloud Bookmarks app: 0.15.1
  • Floccus: 3.2.3
  • Floccus sync method: nextcloud-folders

Steps to reproduce

We are using OTRS. A lot of the users are bookmarking OTRS links to tickets. A direct link to a ticket does look like this: https://my.otrs.dom/index.pl?Action=AgentTicketZoom;TicketID=707128

  1. Bookmark a URL with a ; character. It doesn't matter if this bookmark is created within NextCloud Bookmarks or within your browser.
  2. Let Floccus sync that Bookmark.
  3. Try to open the bookmark within your browser.
  4. The ; character (and all special characters after this one) will be transformed to something like '%3B'.

Expected outcome

The URL shouldn't be transformed by Floccer. Otherwise the links ware mostly likely broken.

Actual outcome

It seems like that Floccer has problems with syncing URLs with ';' in it. With that being said, adding a URL with a ';' character to NextCloud Bookmarks from the webinterface and opening the URL from the webinterface works fine. Using the AddOn FreedomMarks with Firefox also works as expected. However, FreedomMarks does not actually sync the bookmarks rather than accessing them directly through the NextCloud API.

@marcelklehr
Copy link
Member

Thank you for the feedback! This definitely appears to be a bug. I have opened an issue with the library used for URL normalization in floccus and will report back here.

@marcelklehr marcelklehr added this to In progress in Floccus Jan 16, 2019
@marcelklehr
Copy link
Member

I guess I will write my own normalization library...

Floccus automation moved this from In progress to Done Feb 7, 2019
@marcelklehr
Copy link
Member

This should be fixed in the next release

@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
Status: Done
Development

No branches or pull requests

2 participants