algorand / algorand/go-algorand

Convert KMD API Server to new generated framework

Open
#1,814 0 comments 0 reactions 0 assignees View on GitHub
FDE
Dominant language
Go
Stars
1.4k
Forks
537
Avg merge
1d 7h
Merged PRs (30d)
18

Description

## Summary
The algod/indexer v2 servers have a new spec first server layer, we should convert KMD to use the same process. This would allow us to remove all of the build time swagger dependencies, and allow us to use the KMD spec to generate clients as well like we do for some SDKs with the v2 clients.

## Scope/Requirements
Fixup the generated KMD specfile and use oapi-codegen to generate new server code. Replace the existing server with the new one.

There should be no user facing changes in this process to avoid needing to change existing clients at this time.

## Urgency/Relative Priority
This change would allow us to clean up some things:
1. remove the `swagger` tool from build dependency and build. This tool has broken the build many times in the past, and the spec it generates isn't even correct. It needs to go away. (this also depends on removing the algod v1 API). (Dev Ops has to manually fix this every time)
2. makes the HTTP server code more consistent by having a single framework instead of two.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.