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

check for invalid mail objects in user namespace #142

Closed
jrse opened this issue May 17, 2018 · 1 comment
Closed

check for invalid mail objects in user namespace #142

jrse opened this issue May 17, 2018 · 1 comment
Assignees

Comments

@jrse
Copy link
Contributor

jrse commented May 17, 2018

in case rmb tool reads a object which does not represents a valid mail object.
the process exits with -1.
e.g.:

terminate called after throwing an instance of 'std::invalid_argument'

what(): stol

Aborted (core dumped)

jrse added a commit that referenced this issue May 22, 2018
- added rmb-commands.h/cpp 
- added validate metadata function to utils
- added metadata check to rmb tool.
jrse added a commit that referenced this issue May 22, 2018
jrse added a commit that referenced this issue May 22, 2018
jrse added a commit that referenced this issue May 22, 2018
good case all mail metadata set.
jrse added a commit that referenced this issue May 22, 2018
- new unit test to evaluate handling of invalid receive / save date .
jrse added a commit that referenced this issue May 23, 2018
- unit test: skip invalid rados object (no metadata and size < 0)
- moved metadata module creation to rmb-commands.
jrse added a commit that referenced this issue May 23, 2018
- added new unit tests
  read object with metadata good case
  read object with invalid receive date.
jrse added a commit that referenced this issue May 23, 2018
removed some unnecessary log messages and added additional try catch
statements around stol statements.
jrse added a commit that referenced this issue May 23, 2018
jrse added a commit that referenced this issue May 23, 2018
@jrse jrse self-assigned this May 24, 2018
@jrse
Copy link
Contributor Author

jrse commented May 24, 2018

implemented.

@jrse jrse closed this as completed May 24, 2018
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

2 participants