Skip to content

feat(ov_api): add and delete ownership voucher in onboarding server #299

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

Closed
wants to merge 5 commits into from

Conversation

say-paul
Copy link
Contributor

@say-paul say-paul commented Jul 26, 2022

Closes: #132
endpoints:
POST /management/v1/ownership_voucher

POST /management/v1/ownership_voucher/delete

@say-paul say-paul requested a review from puiterwijk July 26, 2022 13:37
@nullr0ute
Copy link
Contributor

nullr0ute commented Jul 26, 2022

This needs at least a decent descriptive commit message, a header with "working model" isn't really enough, What does the patch do? From the commit message I can not tell. Also possibly a sign off line (Not sure if we explicitly require them).

Also tests? They should probably be in a second commit in the PR.

let ov_list: [&str; 1] = ["89cb17fd-95e7-4de8-a36a-686926a7f88f"];
let delete_ov = client
.post(format!(
"http://localhost:{}/management/v1/ownership_voucher/delete",

Check failure

Code scanning / devskim

Insecure URL

Insecure URL
@say-paul say-paul changed the title feat(ov_api): working model feat(ov_api): add and delete ownership voucher in onboarding server Jul 27, 2022
@say-paul say-paul force-pushed the ov_api branch 3 times, most recently from c0d3dcc to 89007e7 Compare July 27, 2022 19:14
@say-paul
Copy link
Contributor Author

Added test and fixed the commit message

ownership voucher management provide easy api to  mange fleet of devices
ov management provides the below feature:

-add ownership voucher to onborading server for easier device onboarding
-supported ov formats are cobr and pem.
-user can also delete device(s) using guid

Signed-off-by: Sayan Paul <[email protected]>
@nullr0ute
Copy link
Contributor

Closing this for the time being as it needs work, we'll revisit when we have a more defined requirement.

@nullr0ute nullr0ute closed this Jan 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement OV Management API
2 participants