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

Rest API Feature: cryptokeys POST #2

Closed
2 tasks done
benj-zen opened this issue May 16, 2016 · 4 comments
Closed
2 tasks done

Rest API Feature: cryptokeys POST #2

benj-zen opened this issue May 16, 2016 · 4 comments
Assignees

Comments

@benj-zen
Copy link
Collaborator

benj-zen commented May 16, 2016

Feature

Implement the POST Method for cryptokeys.
Description: Link

  • Implement feature
  • Implement test
@benj-zen benj-zen self-assigned this May 16, 2016
@benj-zen
Copy link
Collaborator Author

benj-zen commented May 16, 2016

Sadly there is an Issue #706 which prevents me to get the id of the newly added key. This lets my solution become a bit dirty. I'm currently looking for the best way to solve this.

@benj-zen
Copy link
Collaborator Author

c95e734 first implementation

@benj-zen
Copy link
Collaborator Author

Depends on issue #4

@MrM0nkey
Copy link
Collaborator

MrM0nkey commented Jun 9, 2016

@benj-zen Does the ApiException like here closes the methode? And if do the client get a response? It seems that
resp->setErrorResult("Reasons happen", 422); return;
would fit the api-spec better.

MrM0nkey pushed a commit that referenced this issue Jun 17, 2016
Defined testcases for POST.
Added assert_error_json to test_helper.py.
#2 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

2 participants