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

error messages in crypto plugins&tests #3022

Closed
markus2330 opened this issue Oct 2, 2019 · 5 comments · Fixed by #3058

Comments

@markus2330
Copy link
Contributor

commented Oct 2, 2019

I think in general it would be good if the crypto plugins and the crypto test cases give better error messages. Should I open a new issue for this?

 95/267 Test  #93: testmod_gpgme .................................***Failed    0.18 sec
OK closing connection
GPGME        TESTS
==================

/home/markus/Projekte/Elektra/current/src/plugins/gpgme/testmod_gpgme.c:227: error in test_encryption_decryption: kdb get failed
/home/markus/Projekte/Elektra/current/src/plugins/gpgme/testmod_gpgme.c:243: error in test_encryption_decryption: key type not restored to string during decryption
/home/markus/Projekte/Elektra/current/src/plugins/gpgme/testmod_gpgme.c:244: error in test_encryption_decryption: key content not restored during decryption
/home/markus/Projekte/Elektra/current/src/plugins/gpgme/testmod_gpgme.c:254: error in test_encryption_decryption: key content not restored during decryption

gpgme Results: 45 Tests done — 4 errors
@markus2330 markus2330 changed the title testmod_gpgme fails on buster error messages in crypto plugins&tests Oct 2, 2019
@markus2330

This comment has been minimized.

Copy link
Contributor Author

commented Oct 2, 2019

Ok, the problem is that spec is not writable. But this cannot seen by the error messages at all. So this issue is only about improving error messages 👍

@petermax2

This comment has been minimized.

Copy link
Member

commented Oct 2, 2019

OK I will check on this issue.

@markus2330

This comment has been minimized.

Copy link
Contributor Author

commented Oct 2, 2019

Thank you very much! Any other improvement of the error messages (usability) is also very much welcomed!

@petermax2

This comment has been minimized.

Copy link
Member

commented Oct 7, 2019

I'll have a look at

  • fcrypt
  • crypto
  • gpgme

But probably not before the weekend. Let's keep this issue open.

@markus2330

This comment has been minimized.

Copy link
Contributor Author

commented Oct 7, 2019

Thank you very much!

petermax2 added a commit to petermax2/libelektra that referenced this issue Oct 12, 2019
See ElektraInitiative#3022 for full discussion.
petermax2 added a commit to petermax2/libelektra that referenced this issue Oct 12, 2019
See ElektraInitiative#3022 for full discussion.
petermax2 added a commit to petermax2/libelektra that referenced this issue Oct 12, 2019
See ElektraInitiative#3022 for full discussion.
@petermax2 petermax2 referenced this issue Oct 12, 2019
5 of 16 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.