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

Articles duplicates when URI contains non-ascii characters #2126

Closed
Rurik19 opened this issue May 30, 2016 · 5 comments
Closed

Articles duplicates when URI contains non-ascii characters #2126

Rurik19 opened this issue May 30, 2016 · 5 comments
Assignees
Labels
Milestone

Comments

@Rurik19
Copy link
Contributor

Rurik19 commented May 30, 2016

Issue details

Articles duplicates when URI contains non-ascii characters

Environment

  • wallabag version 2.0.4

Steps to reproduce/test case

add param with nonascii chars to url and save -> new article

@j0k3r
Copy link
Member

j0k3r commented May 30, 2016

Could you give us urls example?

@Rurik19
Copy link
Contributor Author

Rurik19 commented May 30, 2016

Voila

this one saves only once
https://habrahabr.ru/company/intel/blog/283066/?aa=asciitext

but this one - as many times as i save it
https://habrahabr.ru/company/intel/blog/283066/?aa=русский

@j0k3r j0k3r added the Bug label May 30, 2016
@j0k3r
Copy link
Member

j0k3r commented Jun 17, 2016

This will be a bit hard to solve.
Could you give me an example where the parameter really mater on the content? I mean, like if the parameter is broken the content will be 404 or completely different?

Thanks

@Rurik19
Copy link
Contributor Author

Rurik19 commented Jun 20, 2016

Could you give me an example where the parameter really mater on the content?

No, I cannot. Parameters don't affect the content, but only influence the search of existing article with same URI. The links above are good example.

@MatElGran
Copy link

http://www.aritylabs.com/post/106091708292/des-contrôleurs-optionnels-grâce-à-constmissing poses problem too. There's a new item created each time I save it.

@j0k3r j0k3r self-assigned this Aug 10, 2016
@j0k3r j0k3r closed this as completed Oct 2, 2016
@j0k3r j0k3r added this to the 2.1.0 milestone Oct 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants