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

record locking is broken #131

Closed
stevepiercy opened this issue Jan 20, 2017 · 2 comments
Closed

record locking is broken #131

stevepiercy opened this issue Jan 20, 2017 · 2 comments

Comments

@stevepiercy
Copy link
Member

stevepiercy commented Jan 20, 2017

Record locking fails with error messages, depending on whether one is trying to edit:

Keyfield or keyvalue missing 7006

or delete:

Record locked by another user 7010

a record in the Knop 9 demo under Customer Advanced > Edit.

In addition, an anonymous user can delete any record with a record lock through the Customers Simple interface.

@stevepiercy stevepiercy added this to the 0.3 milestone Jan 20, 2017
@stevepiercy stevepiercy changed the title getrecord fails with -lock=true record locking is broken getrecord fails with -lock=true Jan 20, 2017
@stevepiercy stevepiercy changed the title record locking is broken getrecord fails with -lock=true record locking is broken Jan 20, 2017
@stevepiercy
Copy link
Member Author

I have no idea why this does not work, but I isolated it to an issue with knop_seed and blowfish_decrypt using an instance variable set to the value of knop_seed.

To replicate

  • Checkout commit 8baeda3
  • Login and navigate to Customers Advanced, click an item, edit it, then try to save it.

Potential fix

  • Change the definition of knop_seed in knop_utils.lasso by checking out commit

Test code used in a failed attempt to replicate the issue

https://gist.github.com/stevepiercy/5b8dd3d67edf7aa61671395d3ea167ab

@stevepiercy
Copy link
Member Author

Fixed in 6e1b054

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant