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.

Sign up for GitHub

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

Jump to bottom

Incorrect glossary import nonce check #673

Closed
ocean90 opened this issue Feb 27, 2017 · 0 comments
Closed

Incorrect glossary import nonce check #673

ocean90 opened this issue Feb 27, 2017 · 0 comments
Assignees
@yoavf
Labels
[Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature is broken.
Milestone
2.3.1

Comments

@ocean90
Copy link
Member

ocean90 commented Feb 27, 2017

As reported by @yoavf:

The path for the default project for the locale is actually //languages (as set by get_locale_glossary_project)
When using project_path from the route, we only get one slash, and the nonce check fails.
Even if it was meant to only have one slash, it is safer to use the path from the project retrieved earlier.
@ocean90 ocean90 added [Type] Bug An existing feature is broken. [Status] In Progress Tracking issues with work in progress labels Feb 27, 2017
@ocean90 ocean90 added this to the 2.3.1 milestone Feb 27, 2017
@yoavf yoavf mentioned this issue Feb 27, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yoavf yoavf

Labels
[Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature is broken.
Projects
None yet
Milestone
2.3.1
Development

No branches or pull requests
2 participants
@ocean90 @yoavf