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

Consignment validation fails with single Stock/Inventory #56

Closed
crisdut opened this issue May 2, 2023 · 4 comments · Fixed by #63
Closed

Consignment validation fails with single Stock/Inventory #56

crisdut opened this issue May 2, 2023 · 4 comments · Fixed by #63
Assignees
Labels
bug Something isn't working
Milestone

Comments

@crisdut
Copy link
Member

crisdut commented May 2, 2023

According to my comment here, I found a bug when I try to validate the consignment file of the second transfer.

Consignment is NOT valid
Validation failures:
- seal 0565da1234fdf84a7f804d9562476f44b65972553bc50895f31bbaec5889a44e/0/0 present in the history is confidential and can't be validated.
Validation warnings:
- duplicated terminal seal FzaZEQgyyvqaqv4EGcLNTqi2YGf7Kx4ZKDnvLRDmD16Q in operation d36bb3980eee63db9e844d2a6dfcf53d95eede4e861f7e1b12412cd1e2729336.

Important: This error occurs only when we create an invoice and a transfer using a single stock/inventory. When we use different persistence stocks, everything works!

@dr-orlovsky
Copy link
Member

Can you please send be the consignment - and also the stock.dat file? That would really help in debugging

@dr-orlovsky dr-orlovsky self-assigned this May 2, 2023
@dr-orlovsky dr-orlovsky added the bug Something isn't working label May 2, 2023
@dr-orlovsky dr-orlovsky added this to the v0.10.x milestone May 2, 2023
@dr-orlovsky
Copy link
Member

I will work on this issue independently from the merge of #36 (comment) since it has lesser criticality

@crisdut
Copy link
Member Author

crisdut commented May 2, 2023

Can you please send be the consignment - and also the stock.dat file? That would really help in debugging

I have attached the information you requested. Furthermore, I kept saving the "state" of each stock as I performed the operations (the name of each stock is self-explanatory).

dump.tar.gz

@dr-orlovsky
Copy link
Member

The fix I found: #63

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

2 participants